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

Update broken links to opensearch docs in README.md #314

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

spapadop
Copy link
Contributor

@spapadop spapadop commented Oct 3, 2024

Description

Updating some broken links to opensearch docs


  • All my commits include DCO.
    DCO stands for Developer Certificate of Origin and it is your declaration that your contribution is correctly attributed and licensed. Please read more about how to attach DCO to your commits here (spoiler alert: in most cases it is as simple as using -s option when doing git commit).
    Please be aware that commits without DCO will cause failure of PR CI workflow and can not be merged.

@lukas-vlcek
Copy link
Collaborator

Thanks @spapadop for doc fix.
Would you mind signing your commit so that CI checks passes? Please see above for details, you basically need to do git commit -s ... or manually put your email into commit description (see any previous commit message for format/example).

Copy link
Collaborator

@lukas-vlcek lukas-vlcek left a comment

Choose a reason for hiding this comment

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

Please sign your commits. git commit -s ...

@spapadop
Copy link
Contributor Author

spapadop commented Oct 4, 2024

Oops sorry, it should be good now.

@lukas-vlcek lukas-vlcek merged commit 4ad34e4 into Aiven-Open:main Oct 4, 2024
3 checks passed
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