Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update spaze/phpstan-disallowed-calls requirement from 2.14.2 to 2.15.0 #58

Merged

Conversation

dependabot[bot]
Copy link

@dependabot dependabot bot commented on behalf of github Jun 1, 2023

Updates the requirements on spaze/phpstan-disallowed-calls to permit the latest version.

Release notes

Sourced from spaze/phpstan-disallowed-calls's releases.

New definedIn & exclude directives to limit selection

Can exclude some attributes, calls, namespaces (#197, #199)

Handy when you disallow items with wildcards but there's this one thing you'd like to leave out.

parameters:
    disallowedFunctionCalls:
        -
            function: 'pcntl_*()'
            exclude:
                - 'pcntl_foo*()'

exclude can be a string or an array/list of strings. Currently works for attributes, function & method calls, namespaces.

Add definedIn?:string|list<string> config option (#198, #200)

To further specify/limit files where the function or method should be defined to be disallowed.

parameters:
    disallowedFunctionCalls:
        -
            function: '*'
            definedIn:
                - 'vendor/foo/bar'
    disallowedMethodCalls:
        -
            method: '*'
            definedIn:
                - 'vendor/foo/bar'

definedIn can also be string or a list/array of strings. Currently works for function and method calls only. You may also need to set filesRootDir, see the README.

Internal changes

  • Internal naming cleanup (#195)
  • Require symfony/polyfill-php80 for dev/tests because some tests use str_starts_with() (#196)
Commits
  • cae3e66 Make the definedIn test actually test what it should test (#200)
  • 5d6389d Make the definedIn test actually test what it should test
  • 12a42e7 Can specify exclude as string (#199)
  • efa81ff Can specify exclude as string
  • ee07fd9 Add definedIn?:string|list\<string> config option (#198)
  • 7c755c4 Add definedIn?:string|list<string> config option to further specify files whe...
  • e9a2fdc Fix typos
  • b49a7fd Can exclude some attributes, calls, namespaces (#197)
  • 114ded7 Can exclude some attributes, calls, namespaces
  • 245c927 A new identifier matching service
  • Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot will merge this PR once CI passes on it, as requested by @xificurk.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Pull requests that update a dependency file php Pull requests that update Php code labels Jun 1, 2023
@xificurk
Copy link

xificurk commented Jun 1, 2023

@dependabot squash and merge

@dependabot dependabot bot force-pushed the dependabot/composer/spaze/phpstan-disallowed-calls-2.15.0 branch from e874abb to 9a27974 Compare June 1, 2023 14:09
@xificurk
Copy link

xificurk commented Jun 1, 2023

@dependabot squash and merge

Updates the requirements on [spaze/phpstan-disallowed-calls](https://github.com/spaze/phpstan-disallowed-calls) to permit the latest version.
- [Release notes](https://github.com/spaze/phpstan-disallowed-calls/releases)
- [Commits](spaze/phpstan-disallowed-calls@v2.14.2...v2.15.0)

---
updated-dependencies:
- dependency-name: spaze/phpstan-disallowed-calls
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/composer/spaze/phpstan-disallowed-calls-2.15.0 branch from 9a27974 to 303e68f Compare June 1, 2023 14:13
@xificurk
Copy link

xificurk commented Jun 1, 2023

@dependabot squash and merge

@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Jun 1, 2023

One of your CI runs failed on this pull request, so Dependabot won't merge it.

Dependabot will still automatically merge this pull request if you amend it and your tests pass.

@xificurk
Copy link

xificurk commented Jun 3, 2023

@dependabot squash and merge

@dependabot dependabot bot merged commit c186caa into master Jun 3, 2023
21 checks passed
@dependabot dependabot bot deleted the dependabot/composer/spaze/phpstan-disallowed-calls-2.15.0 branch June 3, 2023 05:16
@coveralls
Copy link

Pull Request Test Coverage Report for Build 5145395492

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 5145286808: 0.0%
Covered Lines: 0
Relevant Lines: 148

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 5145395492

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 5145286808: 0.0%
Covered Lines: 0
Relevant Lines: 148

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants