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

Spelling #1918

Merged
merged 62 commits into from
Mar 1, 2022
Merged

Spelling #1918

merged 62 commits into from
Mar 1, 2022

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Feb 25, 2022

What type of PR is this?

/kind cleanup
/kind rule-update

Any specific area of the project related to this PR?

/area build
/area engine
/area rules
/area tests
/area proposals

What this PR does / why we need it:

This PR corrects misspellings identified by the check-spelling action.

The misspellings have been reported at jsoref@fe4788d#commitcomment-67427362

The action reports that the changes in this PR would make it happy: jsoref@7978e68

Which issue(s) this PR fixes:

Special notes for your reviewer:

Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.

Does this PR introduce a user-facing change?:

rule(Detect outbound connections to common miner pool ports): fix url in rule output
rule(macro somebody_becoming_themself): renaming macro to somebody_becoming_themselves

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
@poiana
Copy link

poiana commented Feb 25, 2022

Welcome @jsoref! It looks like this is your first PR to falcosecurity/falco 🎉

@poiana poiana added the size/L label Feb 25, 2022
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
### Archivation
### Archiving
Copy link
Contributor Author

Choose a reason for hiding this comment

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

?

- macro: somebody_becoming_themself
- macro: somebody_becoming_themselves
Copy link
Contributor Author

Choose a reason for hiding this comment

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

happy to drop if people have a strong opinion

output: Outbound connection to IP/Port flagged by cryptoioc.ch (command=%proc.cmdline port=%fd.rport ip=%fd.rip container=%container.info image=%container.image.repository)
output: Outbound connection to IP/Port flagged by https://cryptoioc.ch (command=%proc.cmdline port=%fd.rport ip=%fd.rip container=%container.info image=%container.image.repository)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fwiw, this site is dead.

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Copy link
Contributor

@jasondellaluce jasondellaluce left a comment

Choose a reason for hiding this comment

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

/approve

Thank you @jsoref, this is such a great contribution! Would you consider running this on https://github.com/falcosecurity/libs and on https://github.com/falcosecurity/falco-website as well?

@poiana
Copy link

poiana commented Mar 1, 2022

LGTM label has been added.

Git tree hash: 031284be54e7546afb2bf1a32844f23729195b31

@poiana poiana added the approved label Mar 1, 2022
Copy link
Member

@leogr leogr left a comment

Choose a reason for hiding this comment

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

/approve

@poiana
Copy link

poiana commented Mar 1, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jasondellaluce, jsoref, leogr

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@leogr
Copy link
Member

leogr commented Mar 1, 2022

/milestone 0.32.0

@poiana poiana merged commit 7c005aa into falcosecurity:master Mar 1, 2022
@jsoref jsoref deleted the spelling branch March 1, 2022 16:18
@jsoref
Copy link
Contributor Author

jsoref commented Mar 1, 2022

Sure, added to my todo list https://github.com/stars/jsoref/lists/to-check-spelling (which is quite long).

@jasondellaluce jasondellaluce added this to the 0.31.1 milestone Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants