Skip to content

Replace "annotation inject" with "attribute Inject" - #1066

Closed
mildabre wants to merge 151 commits into
nette:masterfrom
mildabre:patch-23
Closed

Replace "annotation inject" with "attribute Inject"#1066
mildabre wants to merge 151 commits into
nette:masterfrom
mildabre:patch-23

Conversation

@mildabre

@mildabre mildabre commented Nov 7, 2024

Copy link
Copy Markdown
Contributor

update to PHP 8.0 - replace annotation with attribute

@dg
dg force-pushed the master branch 29 times, most recently from e0bc7f4 to 3c92c82 Compare December 18, 2024 14:49
@dg

dg commented Jul 5, 2026

Copy link
Copy Markdown
Member

Thanks, you're right that "annotation inject" is outdated — it's the #[Inject] attribute now. Incorporated in 1de725ea8, applied to both the Czech and English versions.

One note: I tested your exact markup [#[Inject] |…] in Texy and it doesn't render as a link — the ] inside #[Inject] prematurely closes the link, so the whole thing stays as literal text (backticks around it don't help either). So I linked the plain word Inject instead ("s atributem [Inject |…]" / "with the [Inject |…] attribute"), which renders correctly and still points to the Inject-attributes section. Closing since the branch can't be merged directly.

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.