Skip to content

Merge updates from original repo#4

Merged
Gruven merged 52 commits intomasterfrom
mergeUpdatesFromOriginalRepo
Mar 10, 2026
Merged

Merge updates from original repo#4
Gruven merged 52 commits intomasterfrom
mergeUpdatesFromOriginalRepo

Conversation

@npogadaev
Copy link
Copy Markdown
Member

Залил изменения из последней версии оригинальной библиотеки. Добавил в новую версию изменения которые были внесены в нашем форке. Внес правки в новый код который появился после слияния изменений

pandabadger and others added 30 commits February 2, 2025 15:13
…g parameter $connection as nullable

```Deprecated: Foolz\SphinxQL\SphinxQL::__construct(): Implicitly marking parameter $connection as nullable is deprecated, the explicit nullable type must be used instead in vendor/foolz/sphinxql-query-builder/src/SphinxQL.php on line 222```
…rameter $next as nullable is deprecated

```
Deprecated: Foolz\SphinxQL\SphinxQL::enqueue(): Implicitly marking parameter $next as nullable is deprecated, the explicit nullable type must be used instead in /vendor/foolz/sphinxql-query-builder/src/SphinxQL.php on line 307
```
…g parameter $connection as nullable is deprecated

Fixes FoolCode#214
Fix PHP 8.4 implicit nullable deprecations
Deprecated: Foolz\SphinxQL\SphinxQL::__construct(): Implicitly markin…
Deprecated: Foolz\SphinxQL\SphinxQL::enqueue(): Implicitly marking pa…
Deprecated: Foolz\SphinxQL\SphinxQL::__construct(): Implicitly markin…
…tions

ci: migrate from Travis to GitHub Actions
@npogadaev npogadaev requested a review from Gruven March 5, 2026 12:06
@npogadaev npogadaev requested a review from Gruven March 6, 2026 10:52
@Gruven Gruven merged commit 77a37df into master Mar 10, 2026
14 checks passed
Gruven added a commit that referenced this pull request Mar 10, 2026
@Gruven
Copy link
Copy Markdown
Member

Gruven commented Mar 10, 2026

Синхронизация была сделана через rebase, из-за чего был лишний мёрж и github не видит синхронизации форка. Также заметил кривой мёрж как минимум в readme. Требуется переделать пул реквест с мёрж, также свои изменения подписать более информативно, чем "Fix". Откатил мастер к предыдущему состоянию.

@npogadaev npogadaev deleted the mergeUpdatesFromOriginalRepo branch March 10, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants