Skip to content

Conversation

@Boegie
Copy link
Contributor

@Boegie Boegie commented Nov 15, 2024

Closes #808

@Boegie
Copy link
Contributor Author

Boegie commented Nov 15, 2024

Currently very much a Work In Progress.

@Boegie
Copy link
Contributor Author

Boegie commented Nov 15, 2024

Done, if we land this before #807, it should even work for both the 1.x and the main branch AFAICT

@mglaman
Copy link
Owner

mglaman commented Nov 19, 2024

I'll merge at the computer so I can cherry pick

@mglaman mglaman merged commit d505a16 into mglaman:main Nov 19, 2024
17 of 18 checks passed
mglaman pushed a commit that referenced this pull request Nov 19, 2024
* [Code] Reworked \mglaman\PHPStanDrupal\Drupal\ExtensionMap::keyByExtensionName

* [HACK] Temporarily remove core_baseline test

* [Code] Reworked \mglaman\PHPStanDrupal\Drupal\ExtensionMap::keyByExtensionName

* [Code] State these @phpstan-ignore-next-line have to stay

* [Code] Remov eBC for PHP-Parser 4.x.

* [Code] Remove BC for PHP-Parser 4.x.

* [Code] Go wild and add an explanation why we want to keep the @phpstan-ignore-next-line in these cases

* [Code] More replacements

* Revert "[HACK] Temporarily remove core_baseline test"

This reverts commit e0b8115.
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.

Replace @phpstan-ignore-next-line with more targeted @phpstan-ignore [identifier].

2 participants