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

Prefer attributes, deprecate annotations #1651

Merged
merged 1 commit into from
Sep 25, 2024

Conversation

fulldecent
Copy link
Contributor

No description provided.

@fulldecent fulldecent changed the title Prefer attributes Prefer attributes, deprecate annotations Sep 22, 2024
@fulldecent
Copy link
Contributor Author

This adds a deprecation notice so we can soon update to only supported PHP versions.


For reference, here are the supported versions of PHP.

Please note that future versions of PHP will have longer support cycles (just for the security support, not the active support), however 7.4 and even 8.0 are specifically excluded.

This is a clear signal to the community not to use 7.4 and 8.0.

README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@DerManoMann DerManoMann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry to be a pain...

README.md Outdated
@@ -19,6 +20,8 @@ For a full list of supported annotations, please have look at the [`OpenApi\Anno
- Exceptional error reporting (with hints, context)
- As of PHP 8.1 all annotations are also available as PHP attributes


Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh, another nit: why these extra lines?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ugggh prettier, that's why :-(

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did rebase squash should be fixed now

@fulldecent
Copy link
Contributor Author

Squash rebase, fix whitespace

@DerManoMann

Copy link
Collaborator

@DerManoMann DerManoMann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not particularly like the (preferred) - the library doesn't really care ;) I suppose it is the more modern approach 🦐

@DerManoMann DerManoMann merged commit ff78ab1 into zircote:master Sep 25, 2024
21 checks passed
@DerManoMann
Copy link
Collaborator

Thanks @fulldecent

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