Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[PHPDoc] Fix various PHPDoc syntax errors#62316

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
yoeunes wants to merge491 commits intosymfony:7.4fromyoeunes:phpdocs-syntax

Conversation

@yoeunes
Copy link
Contributor

QA
Branch?8.0
Bug fix?no
New feature?no
BC breaks?no
Deprecations?no
Issuesn/a
LicenseMIT

This PR fixes several minor PHPDoc syntax errors

xabbuhand others added30 commitsJuly 24, 2025 15:18
* 7.4:  fix merge  add back setAccessible() for PHP 7 compatibility  Update BrevoRequestParser.php  [JsonStreamer] Add `include_null_properties` option  [Translation] Add `StaticMessage`  [Form][PhpUnitBridge] Remove usage of noop `ReflectionProperty::setAccessible()`  fix compatibility with different Relay versions  [VarDumper][Serializer] Remove require php 8.2 from tests  [HttpKernel] Remove outdated phpdoc  [Console] Fix JSON description for negatable input flags  [JsonStreamer] Allow to access current object when writing
* 7.4:  [Serializer] Make data provider return type match its PHPDoc  fix fixture files locations  resolve conflict  [Console] Fix `TreeHelper::addChild` when providing a string  [Serializer] Handle invalid mapping type property type  [Config] Do not generate unreachable configuration paths  [WebProfilerBundle] Fix missing indent on non php files opended in the profiler  [Serializer] Add missing CHANGELOG entry  [Validator] Add `min` and `max` in both error messages of `LengthValidator`  remove docblocks for deprecated argument types  [ObjectMapper] skip reading uninitialized values  [Security] Support union type for `#[CurrentUser]` attribute  Fix typos  [Serializer] Add CDATA_WRAPPING_NAME_PATTERN support to XmlEncoder  [JsonPath] Fix parsing invalid Unicode codepoints  [FrameworkBundle] Fix cache warmers tests  [AssetMapper] Add support for loading JSON using import statements  [BrowserKit] Add PHPUnit constraints: `BrowserHistoryIsOnFirstPage` and `BrowserHistoryIsOnLastPage`
* 7.4:  Turn fabbot into a reusable github action
* 7.4:  [fabbot] Make license check optional
* 7.4:  Leverage symfony-tools/fabbot
* 7.4:  [OptionsResolver] Optimize splitOutsideParenthesis() - 5.9x faster  Use unique identifier for RequestContextProvider  [DependencyInjection] Update `ResolveClassPass` to check class existence  [Validator] Add Japanese translation for Twig template validator  fix doc url  [FrameworkBundle] Fix `lint:container --resolve-env-vars`
* 7.4:  deprecate passing choices as $options argument to Choice constraint
…nt option handling (xabbuh)This PR was merged into the 8.0 branch.Discussion----------[Validator] remove support for generic constraint option handling| Q             | A| ------------- | ---| Branch?       | 8.0| Bug fix?      | no| New feature?  | yes| Deprecations? | no| Issues        || License       | MITopening the PR already even if some polishing needs to be done (and not all tests pass yet) to avoid someone else wasting their time on this topicCommits-------09299fc remove support for generic constraint option handling
* 7.4:  run tests using PHPUnit 11.5  make data provider static  [Lock] Fix using fractional TTLs on all platforms  [Lock] Fixes an issue with PostgreSQL when using fractional TTLs  [Console][Table] Fix unnecessary wrapping  [DependencyInjection] Deprecate registering a service without a class when its id is a non-existing FQCN  disable detecting modified indexes with DBAL 4.3  [cache] RedisTrait::doFetch should use pipeline instead of mget for Relay\Cluster  relax assertions for forward compatibility with Symfony 7.4  [Console][Table] Fix invalid UTF-8 due to text wrapping
* 7.4:  account for error message changes in PHP 8.5  account for error message changes in PHP 8.5  [Form] Fix interface name in Guess documentation  Bump Symfony version to 7.3.3  Update VERSION for 7.3.2  Update CHANGELOG for 7.3.2  Bump Symfony version to 6.4.25  Update VERSION for 6.4.24  Update CONTRIBUTORS for 6.4.24  Update CHANGELOG for 6.4.24  [DependencyInjection] Escape parameters before resolving env placeholders  [Validator] review Persian translation for Twig template
* 7.4:  [FrameworkBundle] Escape parameters when serializing a ContainerBuilder  Fix tests sensitive to SYMFONY_IDE env var
* 7.4:  fix low deps tests
* 7.4:  replace PHPUnit annotations with attributes
…s to interfaces as planned (nicolas-grekas)This PR was merged into the 8.0 branch.Discussion----------[Ldap][Messenger][Validator][Workflow] Add methods to interfaces as planned| Q             | A| ------------- | ---| Branch?       | 8.0| Bug fix?      | no| New feature?  | no| Deprecations? | no| Issues        | -| License       | MITCommits-------b428c43 [Ldap][Messenger][Validator][Workflow] Add methods to interfaces as planned
* 7.4:  Fix inline var annotations  run tests with PHPUnit 12.0 on PHP >= 8.3  fix expected stream to native value transformers  replace #[TestWithJson] with #[TestWith]  [Console][Table] Don't split grapheme clusters  [FrameworkBundle] Fix block type from `OK` to `ERROR` when local vault is disabled in `SecretsGenerateKeysCommand`  [FrameworkBundle] Add tests for `secrets:decrypt-to-local`, `encrypt-from-local`, and `generate-keys` commands  Reflection*::setAccessible() has no effect as of PHP 8.1
* 7.4:  Remove some unneeded var annotations  [FrameworkBundle] Decouple ControllerResolverTest from HttpKernel
…ons (xabbuh)This PR was merged into the 8.0 branch.Discussion----------use PHPUnit 11.5 when we need to ignore deprecations| Q             | A| ------------- | ---| Branch?       | 8.0| Bug fix?      | no| New feature?  | no| Deprecations? | no| Issues        || License       | MITWe needsebastianbergmann/phpunit#6239 to be able to use the `--do-not-fail-on-deprecation` option (so either 11.5 or 12.2+). Let's stick with 11.5 for now until our test suite is ready for 12.2 or 12.3.Commits-------5d67f96 use PHPUnit 11.5 when we need to ignore deprecations
* 7.4:  Cleanup .php-cs-fixer.dist.php
* 7.4:  Use phpunit attribute  Remove some implicit bool type juggling  [Scheduler] Fix `scheduler.task` tag arguments optionality  use false instead of null to hide the currency symbol  [FrameworkBundle] Fix block type from `OK` to `ERROR` when local vault is disabled in `SecretsRemoveCommand`  run tests with PHPUnit 12.1 on PHP >= 8.3  Fix wrong boolean values  minorsymfony#61328 [FrameworkBundle] Decouple ControllerResolverTest from HttpKernel (nicolas-grekas)  [Messenger] Fix NoAutoAckStamp handling in Worker::flush()  [DependencyInjection] Dump XML using plain PHP, no DOM needed
* 7.4:  [Form] Fix code style issues  [SecurityBundle] Add tests for `debug:firewall` command  Adding more helpful error messages to the Questionhelper  [Messenger] Allow SQS to handle it's own retry/DLQ
* 7.4:  [ObjectMapper] do not require mapping a target's required promoted property when not on source (#2)  run tests with PHPUnit 12.3  [GitHub] Update .github/PULL_REQUEST_TEMPLATE.md to remove SF 7.2 as it's not supported anymore  [CI] fixed the Intl data tests actions Tests fails due to unknown option -v which is shorthand of the --verbose as per the phpunit its removed without any replacementsebastianbergmann/phpunit#5647 (comment)  [Mailer] Fix expected exception message to include quotes around "http(s)://"  [WebProfilerBundle] Fix toolbar not rendering after replacing it  [Validator] (60455) Validate translations for Arabic (ar)  (60474) Remove translation state attribute for Twig template validator in Ukrainian translation  [VarDumper] Fix dumping objects from the DOM extension  [Mailer] Add MicrosoftGraph API Transport  [Yaml] Fix code style  [Tests] Adapt testAddHtmlContentWithErrors to be HTML5 compliant  Add friendly name in the `to` field  [ObjectMapper] read source metadata before transform  [HtmlSanitizer] Fix force_attributes not replacing existing attribute in initial data
* 7.4:  [HtmlSanitizer] Use the native HTML5 parser when using PHP 8.4+
GromNaNand others added12 commitsNovember 2, 2025 09:09
…dPrevot)This PR was squashed before being merged into the 8.0 branch.Discussion----------[Lock][DynamoDB] Allow current symfony/lock| Q             | A| ------------- | ---| Branch?       | 8.0| Bug fix?      | yes| New feature?  | no| Deprecations? | no| License       | MITFix the version constraint to allow installation of the DynamoDB bridge with the latest Lock component.Commits-------8a981a7 [Lock][DynamoDB] Allow current symfony/lock
* 7.4:  [Lock][DynamoDB] Allow symfony/lock 8.0  [ProxyManager] Remove extra space before return‑type colon in `proxy-implem.php`  [Config] Use the empty string instead of null as an array offset  [Messenger] Support signing messages per handler
* 7.4:  [HttpFoundation] Deprecate null $format argument to Request::setFormat()
* 7.4:  fix test on Windows  simplify test  Bump Symfony version to 7.4.0  Update VERSION for 7.4.0-BETA2  Update CHANGELOG for 7.4.0-BETA2
* 7.4: (21 commits)  [ObjectMapper] lazy loading  [Filesystem] Unify logic for isAbsolute() in Path  [DependencyInjection] Include return type in AppReference shape  [Finder] Make method calls explicit in ExcludeDirectoryFilterIterator  [Cache] Remove unset call on undefined variable in PhpArrayAdapter  [Twig] Ensure WrappedTemplatedEmail::getReturnPath() returns a string  [Routing] Simplify importing routes defined on controller services  Fix tests  [DependendcyInjection] Improve shape for "from_callable" definitions  [PHPDoc] Fix various PHPDoc syntax errors  [Console] Add missing VERBOSITY_SILENT case in CommandDataCollector  [Notifier] Remove unused $transportName argument in EmailChannel::notify()  [Translation] Remove an unused argument passed to parseNode() method  [HttpClient] Reject 3xx pushed responses  [Serializer] Use Asia/Tokyo instead of Japan in tests  [ProxyManagerBridge] Remove comment that reference github discussion  [JsonPath] Remove unused "nothing" property from JsonCrawler  [ErrorHandler] Improve PHPDoc precision in SerializerErrorRenderer  [Routing] Fix matching the "0" URL  [Form] Fix EnumType choice_label logic for grouped choices  ...
@carsonbotcarsonbot added this to the8.0 milestoneNov 5, 2025
@nicolas-grekas
Copy link
Member

8.0? Not lower?

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@xabbuhxabbuhAwaiting requested review from xabbuhxabbuh is a code owner

@lyrixxlyrixxAwaiting requested review from lyrixxlyrixx is a code owner

@dunglasdunglasAwaiting requested review from dunglasdunglas is a code owner

@ycerutoycerutoAwaiting requested review from ycerutoyceruto is a code owner

@welcoMatticwelcoMatticAwaiting requested review from welcoMatticwelcoMattic is a code owner

@kbondkbondAwaiting requested review from kbondkbond is a code owner

@chalasrchalasrAwaiting requested review from chalasrchalasr is a code owner

@OskarStarkOskarStarkAwaiting requested review from OskarStarkOskarStark is a code owner

@jderussejderusseAwaiting requested review from jderussejderusse is a code owner

@alexandre-dauboisalexandre-dauboisAwaiting requested review from alexandre-dauboisalexandre-daubois is a code owner

Assignees

No one assigned

Projects

None yet

Milestone

8.0

Development

Successfully merging this pull request may close these issues.

15 participants

@yoeunes@nicolas-grekas@carsonbot@xabbuh@alexandre-daubois@mttsch@fabpot@santysisi@valtzu@GromNaN@PierreCapel@javiereguiluz@HypeMC@mtarld@DavidPrevot

[8]ページ先頭

©2009-2025 Movatter.jp