Skip to content

Conversation

TysonAndre
Copy link
Contributor

Add a note on something that looks like an invalid array index

Also, see the note on #612 (comment) (This issue (and other issues) are possible to detect in CI, even for uses in inline doc comments such as /** @var T */ $x = some_expr();)

@codecov
Copy link

codecov bot commented Jul 25, 2018

Codecov Report

Merging #625 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #625   +/-   ##
=========================================
  Coverage     82.17%   82.17%           
  Complexity      934      934           
=========================================
  Files            44       44           
  Lines          2154     2154           
=========================================
  Hits           1770     1770           
  Misses          384      384
Impacted Files Coverage Δ Complexity Δ
src/Client/TextDocument.php 100% <ø> (ø) 4 <0> (ø) ⬇️
src/Definition.php 100% <ø> (ø) 6 <0> (ø) ⬇️
src/SignatureHelpProvider.php 98.66% <ø> (ø) 26 <0> (ø) ⬇️
src/PhpDocument.php 79.31% <ø> (ø) 24 <0> (ø) ⬇️
src/Index/Index.php 77.41% <ø> (ø) 54 <0> (ø) ⬇️
src/Cache/ClientCache.php 0% <ø> (ø) 3 <0> (ø) ⬇️

Add a note on something that looks like an invalid array index
@felixfbecker felixfbecker merged commit 450116e into felixfbecker:master Nov 11, 2018
@felixfbecker
Copy link
Owner

🎉 This PR is included in version 5.4.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

2 participants