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

Typo: Changing functions section references $filters instead of $functions #2799

Merged
merged 1 commit into from Sep 21, 2023

Conversation

niclm
Copy link
Contributor

@niclm niclm commented Sep 15, 2023

Issue

Typo in the Changing functions section of Extending Twig docs. References $filters instead of $functions

@niclm niclm requested a review from gchtr as a code owner September 15, 2023 10:14
@coveralls
Copy link

Coverage Status

coverage: 88.522%. remained the same when pulling d855f39 on niclm:patch-1 into 12cff73 on timber:2.x-docs-rc.

@gchtr gchtr deleted the branch timber:2.x September 21, 2023 19:22
@gchtr gchtr closed this Sep 21, 2023
@gchtr
Copy link
Member

gchtr commented Sep 21, 2023

Huh, I didn’t want to close this! Sorry for that, especially because it’s your first pull request!

It happened because you made that pull request against the 2.x-doc-rc branch, which I just deleted by merging in another pull request. I’ll see how I can fix this, because this is a very good catch and we’ll definitely want your pull request to be merged as well.

@gchtr gchtr reopened this Sep 21, 2023
@gchtr gchtr changed the base branch from 2.x-docs-rc to 2.x September 21, 2023 19:29
@gchtr
Copy link
Member

gchtr commented Sep 21, 2023

Alright, I fixed it by readding the branch and the basing it on the 2.x branch. All good. Now I can merge it in 😎.

Thanks a lot @niclm for taking the time to create your first pull request 🎉. Details like these matter a lot!

@gchtr gchtr merged commit 6ef7f69 into timber:2.x Sep 21, 2023
26 checks passed
@niclm
Copy link
Contributor Author

niclm commented Sep 21, 2023

🥳 no problem!

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.

None yet

3 participants