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

templates: fix some typos #15

Closed
wants to merge 1 commit into from

Conversation

invidian
Copy link

Found using 'codespell -f -H -L chage,creat,aks'.

If you like, I can also configure a GH Action running the command above.

Signed-off-by: Mateusz Gozdek mgozdek@microsoft.com

Found using 'codespell -f -H -L chage,creat,aks'.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
@juju4
Copy link
Owner

juju4 commented Mar 26, 2022

LGTM and fine to me for github action

but those files are mirrored from upstream https://github.com/falcosecurity/falco/blob/master/rules/falco_rules.yaml
I just resync those (b880614). It should address those.

If you see other typos, I think that it will better to report to upstream project else change may be reverted by mistake.

@invidian
Copy link
Author

Ah sorry, I didn't know there is another upstream. I'll send the patch there instead. Will close the PR once another is open if you don't mind.

@invidian
Copy link
Author

Opened falcosecurity/falco#1961 instead, but indeed it seems more typos has already been fixed! Thanks.

@invidian invidian closed this Mar 28, 2022
@invidian invidian deleted the invidian/fix-typos branch March 28, 2022 11:36
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