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

Revert "docs: fix Rule spec document typos" #24418

Merged
merged 1 commit into from Mar 17, 2023
Merged

Conversation

aditighag
Copy link
Member

This reverts commit a97f1f4.

The linters should have failed on the PR, but for some reason they didn't.

/cc @nrnrk Please run make generate-k8s-api locally before submitting the changes.

See the failure on other runs - https://github.com/cilium/cilium/actions/runs/4440944264/jobs/7795725126?pr=23831

@aditighag aditighag requested a review from a team as a code owner March 16, 2023 20:14
@maintainer-s-little-helper
Copy link

Commit 0e403b85c8a7f2a383212e8e8a9ffd55255be176 does not contain "Signed-off-by".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Mar 16, 2023
This reverts commit a97f1f4.

Signed-off-by: Aditi Ghag <aditi@cilium.io>
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. label Mar 16, 2023
@aditighag aditighag added dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. release-note/misc This PR makes changes that have no direct user impact. labels Mar 16, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 16, 2023
@aditighag aditighag removed the dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. label Mar 16, 2023
@tklauser tklauser added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 17, 2023
@aanm aanm merged commit 9fb6d7f into cilium:master Mar 17, 2023
41 checks passed
@nrnrk
Copy link
Contributor

nrnrk commented Mar 18, 2023

Sorry to bother you. I recreated this PR.

The CI (generate k8s api) worked this time and it was successful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants