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

docs: Document update-cmdref make target usage #14925

Merged
merged 1 commit into from
Feb 11, 2021

Conversation

nebril
Copy link
Member

@nebril nebril commented Feb 10, 2021

This make target needs to be used by contributors who make changes to
CLI.

@nebril nebril requested review from a team as code owners February 10, 2021 13:18
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Feb 10, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Feb 10, 2021
@pchaigno pchaigno added release-note/misc This PR makes changes that have no direct user impact. sig/contributing Impacts contribution workflow, guidelines, and tools. labels Feb 10, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Feb 10, 2021
@pchaigno pchaigno added the area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. label Feb 10, 2021
@pchaigno pchaigno removed their assignment Feb 10, 2021
@nebril nebril force-pushed the pr/fix-update-cmdref branch 2 times, most recently from 05693d0 to ae0f61d Compare February 10, 2021 13:44
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

This is CI, more about submitting a change than development itself. Should this go into https://docs.cilium.io/en/latest/contributing/development/contributing_guide/#submitting-a-pull-request instead?

Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
@nebril
Copy link
Member Author

nebril commented Feb 10, 2021

@qmonnet good point, will move it to "Submitting a pull request"

@nebril nebril force-pushed the pr/fix-update-cmdref branch 2 times, most recently from 0ed3ce0 to 7e7694b Compare February 11, 2021 12:19
@nebril nebril requested a review from qmonnet February 11, 2021 12:31
This make target needs to be used by contributors who make changes to
CLI.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Thanks!

@qmonnet qmonnet added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 11, 2021
@qmonnet qmonnet removed their assignment Feb 11, 2021
@nathanjsweet nathanjsweet merged commit 8650594 into master Feb 11, 2021
@nathanjsweet nathanjsweet deleted the pr/fix-update-cmdref branch February 11, 2021 20:05
@joestringer joestringer moved this from In progress to Done in 1.10.0 Feb 12, 2021
@joestringer joestringer moved this from In progress to Done in 1.10.0 Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. 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. sig/contributing Impacts contribution workflow, guidelines, and tools.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants