Skip to content

Fix minor spelling mistakes #14190

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

max-chaban
Copy link

@max-chaban max-chaban commented Jun 10, 2025

Proposed commit message

Fixed minor spelling mistakes

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

@max-chaban max-chaban self-assigned this Jun 10, 2025
@max-chaban max-chaban requested a review from a team as a code owner June 10, 2025 08:09
@max-chaban max-chaban added the bug Something isn't working, use only for issues label Jun 10, 2025
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Jun 10, 2025

🚀 Benchmarks report

Package github 👍(5) 💚(0) 💔(1)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
code_scanning 4366.81 3508.77 -858.04 (-19.65%) 💔

To see the full report comment with /test benchmark fullreport

@max-chaban max-chaban force-pushed the fix/spelling-in-github-security-advisories branch from 64a12fa to 3c9230f Compare June 10, 2025 08:40
@max-chaban max-chaban changed the title Fix minor spelling mistake in GitHub Security Advisories Fix minor spelling mistakes Jun 10, 2025
@andrewkroh andrewkroh added Integration:miniflux Miniflux RSS reader Integration:github GitHub Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Jun 10, 2025
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@andrewkroh
Copy link
Member

The two changed packages need a changelog entry and patch version bump. You can use elastic-package changelog add to accomplish this. For example

elastic-package -C packages/github changelog add --next patch --type bugfix --link "https://github.com/elastic/integrations/pull/14190" --description '<changelog text here>'

elastic-package -C packages/miniflux changelog add --next patch --type bugfix --link "https://github.com/elastic/integrations/pull/14190" --description '<changelog text here>'

@max-chaban
Copy link
Author

The two changed packages need a changelog entry and patch version bump. You can use elastic-package changelog add to accomplish this. For example

elastic-package -C packages/github changelog add --next patch --type bugfix --link "https://github.com/elastic/integrations/pull/14190" --description '<changelog text here>'

elastic-package -C packages/miniflux changelog add --next patch --type bugfix --link "https://github.com/elastic/integrations/pull/14190" --description '<changelog text here>'

Ok, thank you, I will add changes today

@max-chaban max-chaban force-pushed the fix/spelling-in-github-security-advisories branch 2 times, most recently from a3ceed2 to 9a7513e Compare June 11, 2025 08:20
Copy link
Contributor

@kcreddy kcreddy left a comment

Choose a reason for hiding this comment

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

Nits. LGTM otherwise

@max-chaban max-chaban force-pushed the fix/spelling-in-github-security-advisories branch from 5fd84e4 to 840ae38 Compare June 11, 2025 11:03
@elasticmachine
Copy link

💚 Build Succeeded

History

cc @max-chaban

Copy link

@andrewkroh andrewkroh added bugfix Pull request that fixes a bug issue and removed bug Something isn't working, use only for issues labels Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull request that fixes a bug issue Integration:github GitHub Integration:miniflux Miniflux RSS reader Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants