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

Broken link in readme to pimple #3527

Closed
artfulrobot opened this issue Feb 7, 2022 · 3 comments
Closed

Broken link in readme to pimple #3527

artfulrobot opened this issue Feb 7, 2022 · 3 comments

Comments

@artfulrobot
Copy link

The link https://pimple.sensiolabs.org/ is dead. In fact it looks like Pimple is ... Well "not accepting changes" according to:

GitHub - silexphp/Pimple: A small PHP dependency injection container - https://github.com/silexphp/Pimple

Which I assume to be the same?

(Im new here: Does grav still rely on pimple?)

Anyway, readme needs updating.

@w00fz
Copy link
Member

w00fz commented Feb 7, 2022

Thanks for letting us know, can you open a PR about it?

mahagr added a commit that referenced this issue Feb 8, 2022
@mahagr
Copy link
Member

mahagr commented Feb 8, 2022

Pimple is so simple that it hasn't really changed for years except for compatibility fixes to PHP and PSR container support.

Fixed the link, thanks!

@mahagr mahagr closed this as completed Feb 8, 2022
@artfulrobot
Copy link
Author

Fair enough! Thanks.

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

No branches or pull requests

3 participants