recommand using the psalm-plugin package over builtin integration (#170)

This commit is contained in:
Saif Eddin Gmati 2021-03-27 20:10:18 +01:00 committed by GitHub
parent a521ed909b
commit 28eb34e788
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 24 additions and 33 deletions

View File

@ -16,6 +16,7 @@ Psl is a standard library for PHP, inspired by [hhvm/hsl](https://github.com/hhv
The goal of Psl is to provide a consistent, centralized, well-typed set of APIs for PHP programmers. The goal of Psl is to provide a consistent, centralized, well-typed set of APIs for PHP programmers.
## Example ## Example
```php ```php
<?php <?php
@ -43,26 +44,13 @@ foo([95, 96, null, 98]);
Supported installation method is via [composer](https://getcomposer.org): Supported installation method is via [composer](https://getcomposer.org):
```console ```shell
$ composer require azjezz/psl composer require azjezz/psl
``` ```
### Psalm Integration ### Psalm Integration
PSL comes with a [Psalm](https://psalm.dev/) plugin, that improves return type for PSL functions that psalm cannot infer from source code. Please refer to the [`php-standard-library/psalm-plugin`](https://github.com/php-standard-library/psalm-plugin) repository.
To enable the Psalm plugin, add the `Psl\Integration\Psalm\Plugin` class to your psalm configuration file plugins list as follows:
```xml
<psalm>
...
<plugins>
...
<pluginClass class="Psl\Integration\Psalm\Plugin" />
</plugins>
</psalm>
```
## Documentation ## Documentation
@ -70,35 +58,32 @@ You can read through the API documentation in [`docs/README.md`](./docs/README.m
## Principles ## Principles
- All functions should be typed as strictly as possible - All functions should be typed as strictly as possible
- The library should be internally consistent - The library should be internally consistent
- References may not be used - References may not be used
- Arguments should be as general as possible. For example, for `array` functions, prefer `iterable` inputs where practical, falling back to `array` when needed. - Arguments should be as general as possible. For example, for `array` functions, prefer `iterable` inputs where practical, falling back to `array` when needed.
- Return types should be as specific as possible - Return types should be as specific as possible
- All files should contain `declare(strict_types=1);` - All files should contain `declare(strict_types=1);`
## Consistency Rules ## Consistency Rules
This is not exhaustive list. This is not exhaustive list.
- Functions argument order should be consistent within the library - Functions argument order should be consistent within the library
- All iterable-related functions take the iterable as the first argument ( e.g. `Iter\map` and `Iter\filter` ) - All iterable-related functions take the iterable as the first argument ( e.g. `Iter\map` and `Iter\filter` )
- `$haystack`, `$needle`, and `$pattern` are in the same order for all functions that take them - `$haystack`, `$needle`, and `$pattern` are in the same order for all functions that take them
- Functions should be consistently named. - Functions should be consistently named.
- If an operation can conceivably operate on either keys or values, the default is to operate on the values - the version that operates on keys should have `_key` suffix (e.g. `Iter\last`, `Iter\last_key`, `Iter\contains`, `Iter\contains_key` ) - If an operation can conceivably operate on either keys or values, the default is to operate on the values - the version that operates on keys should have `_key` suffix (e.g. `Iter\last`, `Iter\last_key`, `Iter\contains`, `Iter\contains_key` )
- Find-like operations that can fail should return `?T`; a second function should be added with an `x` suffix that uses an invariant to return `T` (e.g. `Arr\last`, `Arr\lastx`) - Iterable functions that do an operation based on a user-supplied keying function for each element should be suffixed with `_by` (e.g. `Arr\sort_by`, `Iter\group_by`, `Math\max_by`)
- Iterable functions that do an operation based on a user-supplied keying function for each element should be suffixed with `_by` (e.g. `Arr\sort_by`, `Iter\group_by`, `Math\max_by`)
## Sponsors ## Sponsors
Thanks to our sponsors and supporters: Thanks to our sponsors and supporters:
| JetBrains | | JetBrains |
|---| |---|
| <a href="https://www.jetbrains.com/?from=PSL ( PHP Standard Library )" title="JetBrains" target="_blank"><img src="https://res.cloudinary.com/azjezz/image/upload/v1599239910/jetbrains_qnyb0o.png" height="120" /></a> | | <a href="https://www.jetbrains.com/?from=PSL ( PHP Standard Library )" title="JetBrains" target="_blank"><img src="https://res.cloudinary.com/azjezz/image/upload/v1599239910/jetbrains_qnyb0o.png" height="120" /></a> |
## License ## License
The MIT License (MIT). Please see [`LICENSE`](./LICENSE) for more information. The MIT License (MIT). Please see [`LICENSE`](./LICENSE) for more information.

View File

@ -22,7 +22,8 @@
"friendsofphp/php-cs-fixer": "^2.18", "friendsofphp/php-cs-fixer": "^2.18",
"roave/security-advisories": "dev-master", "roave/security-advisories": "dev-master",
"squizlabs/php_codesniffer": "^3.5", "squizlabs/php_codesniffer": "^3.5",
"vimeo/psalm": "dev-master" "vimeo/psalm": "dev-master",
"php-standard-library/psalm-plugin": "^1.0"
}, },
"autoload": { "autoload": {
"psr-4": { "psr-4": {

View File

@ -8,6 +8,11 @@ use Psalm\Plugin\PluginEntryPointInterface;
use Psalm\Plugin\RegistrationInterface; use Psalm\Plugin\RegistrationInterface;
use SimpleXMLElement; use SimpleXMLElement;
/**
* @deprecated use `php-standard-library/psalm-plugin` package instead.
*
* @see https://github.com/php-standard-library/psalm-plugin
*/
final class Plugin implements PluginEntryPointInterface final class Plugin implements PluginEntryPointInterface
{ {
public function __invoke(RegistrationInterface $registration, ?SimpleXMLElement $config = null): void public function __invoke(RegistrationInterface $registration, ?SimpleXMLElement $config = null): void