Skip to content

Commit

Permalink
Merge 4.x into 5.x
Browse files Browse the repository at this point in the history
  • Loading branch information
SonataCI committed Feb 6, 2022
2 parents e24099f + c4a7897 commit 5905a6e
Show file tree
Hide file tree
Showing 8 changed files with 12 additions and 16 deletions.
1 change: 0 additions & 1 deletion .github/workflows/documentation.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- cron: '30 0 * * *'
push:
branches:
- 3.x
- 4.x
- 5.x
pull_request:
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- cron: '30 0 * * *'
push:
branches:
- 3.x
- 4.x
- 5.x
pull_request:
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/qa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- cron: '30 0 * * *'
push:
branches:
- 3.x
- 4.x
- 5.x
pull_request:
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/symfony-lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- cron: '30 0 * * *'
push:
branches:
- 3.x
- 4.x
- 5.x
pull_request:
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- cron: '30 0 * * *'
push:
branches:
- 3.x
- 4.x
- 5.x
pull_request:
Expand Down
2 changes: 0 additions & 2 deletions .symfony.bundle.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,10 @@
# It's auto-generated by sonata-project/dev-kit package.

branches:
- 3.x
- 4.x
- 5.x

maintained_branches:
- 3.x
- 4.x
- 5.x

Expand Down
20 changes: 12 additions & 8 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -94,19 +94,23 @@ Please note that we try to keep phpdoc to a minimum, so if an `@param` phpdoc
comment brings nothing more than the type hint and variable name already do,
it SHOULD be removed. Descriptions are OPTIONAL if you want to document a type.

If you want to use pseudo-types, generics, array shapes or templates for
static-analysis, prefer the usage of `@phpstan-` annotations over classic
annotations to keep IDE support, and over `@psalm-` annotations for consistency.
Keep standard annotations if necessary.
If you want to use templates for static-analysis, prefer the usage of
`@phpstan-` annotations over classic annotations to keep IDE support, and
over `@psalm-` annotations for consistency. Keep standard annotations if necessary.

```php
/**
* @param Bar|Baz $foo
* @param int $limit a crucial, highly interesting comment
* @param Bar|Baz $foo
* @param class-string $class
* @param int $limit a crucial, fascinating comment
*
* @phpstan-param class-string $class
* @return object
*
* @phpstan-template T of object
* @phpstan-param class-string<T> $class
* @phpstan-return T
*/
protected function bar($foo, string $class, int $limit)
protected function bar($foo, string $class, int $limit): object
{
// ...
}
Expand Down
1 change: 0 additions & 1 deletion docs/.doctor-rst.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,4 +44,3 @@ rules:
whitelist:
lines:
- '.. versionadded:: 4.x'
- 'resource: "@NelmioApiDocBundle/Resources/config/routing.yml"'

0 comments on commit 5905a6e

Please sign in to comment.