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

composer(deps-dev): bump vimeo/psalm from 4.7.3 to 4.8.1 #756

Merged
merged 1 commit into from
Jun 24, 2021

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jun 21, 2021

Bumps vimeo/psalm from 4.7.3 to 4.8.1.

Release notes

Sourced from vimeo/psalm's releases.

Fix baseline functionality

Release 4.8.0 introduced a bug in baseline generation. This release should fix it!

Detect more unused code

Features

New literal-string type

Inspired by the is_literal RFC we've added a new literal-string type.

The type will be most useful to annotate functions and methods that take SQL. In those methods you generally don't want any code that is not part of your app, e.g.

<?php
/** @​param literal-string $sql */
function execute_sql(string $sql, array $params = []): void { }
$id = (string) ($_GET['id'] ?? '');
// passes type checks
execute_sql(
'SELECT * FROM foo WHERE id = :id',
[':id' => $id]
);
// fails
execute_sql(
'SELECT * FROM foo WHERE id = "' .$id . '"'
);

Psalm’s taint analysis can also help detect this general class of issues, but a literal-string type allows the type-checker to provide guarantees much earlier in the process.

Read more in the documentation.

More specific taint analysis for unescaped quotes

htmlentities can be used to strip some harmful characters in strings, but not all of them (by default).

Psalm has a new issue TaintedTextWithQuotes to help detect strings that might not have HTML tags but can have harmful Javascript.

Detect unused return values

When running with the --find-unused-code flag, Psalm already detects a lot of unnecessary code, including unused public and private methods, unused properties and unused variables.

Now Psalm will also flag unused return values — where a function returns something, but nowhere that calls the given function actually uses the returned value.

See the documentation for UnusedReturnValue and PossiblyUnusedReturnValue for more information.

... (truncated)

Commits

Dependabot compatibility score

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 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)

Bumps [vimeo/psalm](https://github.com/vimeo/psalm) from 4.7.3 to 4.8.1.
- [Release notes](https://github.com/vimeo/psalm/releases)
- [Commits](vimeo/psalm@4.7.3...4.8.1)

---
updated-dependencies:
- dependency-name: vimeo/psalm
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@codecov
Copy link

codecov bot commented Jun 21, 2021

Codecov Report

Merging #756 (c46c4a2) into main (d469a15) will not change coverage.
The diff coverage is n/a.

❗ Current head c46c4a2 differs from pull request most recent head 6d55935. Consider uploading reports for the commit 6d55935 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##               main     #756   +/-   ##
=========================================
  Coverage     83.96%   83.96%           
  Complexity       57       57           
=========================================
  Files             4        4           
  Lines           237      237           
=========================================
  Hits            199      199           
  Misses           38       38           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d469a15...6d55935. Read the comment docs.

@localheinz localheinz self-assigned this Jun 24, 2021
@localheinz localheinz merged commit 11a7b98 into main Jun 24, 2021
@localheinz localheinz deleted the dependabot/composer/vimeo/psalm-4.8.1 branch June 24, 2021 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant