Skip to content

Add developer documentation#126

Merged
kodiakhq[bot] merged 1 commit intomainfrom
setup-docs
Oct 14, 2022
Merged

Add developer documentation#126
kodiakhq[bot] merged 1 commit intomainfrom
setup-docs

Conversation

@rgreinho
Copy link
Copy Markdown
Collaborator

@rgreinho rgreinho commented Oct 5, 2022

Adds a documentation page explaining how to setup a developer
environment.

Signed-off-by: Rémy Greinhofer remy.greinhofer@gmail.com

Copy link
Copy Markdown
Contributor

@lumjjb lumjjb left a comment

Choose a reason for hiding this comment

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

Awesome!!! this is super helpful and will let folks try it out real easy!!

Comment thread SETUP.md
Comment thread SETUP.md
@rgreinho rgreinho marked this pull request as ready for review October 7, 2022 18:10
Comment thread SETUP.md
Comment thread SETUP.md Outdated
Comment thread SETUP.md Outdated
@lumjjb lumjjb added the needs-review Needs writer LGTM label Oct 8, 2022
@rgreinho rgreinho force-pushed the setup-docs branch 2 times, most recently from c8263a0 to 6d1503f Compare October 11, 2022 16:16
Comment thread Makefile
@mlieberman85
Copy link
Copy Markdown
Collaborator

Minor nit, and then I think it's good.

Adds a documentation page explaining how to setup a developer
environment.

Signed-off-by: Rémy Greinhofer <remy.greinhofer@gmail.com>
Copy link
Copy Markdown
Collaborator

@pxp928 pxp928 left a comment

Choose a reason for hiding this comment

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

LGTM

@kodiakhq kodiakhq Bot merged commit debb811 into main Oct 14, 2022
@kodiakhq kodiakhq Bot deleted the setup-docs branch October 14, 2022 13:25
mrizzi added a commit to mrizzi/guac that referenced this pull request Sep 18, 2024
Co-authored-by: Dejan Bosanac <dbosanac@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Needs writer LGTM size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants