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

updated Jenkins on AWS tutorial documentation #5629

Merged
merged 3 commits into from Oct 28, 2022

Conversation

paulmarsicloud
Copy link
Contributor

  • Updated the older AWS Console UI images to the newer version
  • Added step by step instructions for Configure Clouds as well

@paulmarsicloud paulmarsicloud requested a review from a team as a code owner October 26, 2022 01:48
@probot-autolabeler probot-autolabeler bot added the documentation Jenkins documentation, including user and developer docs, solution pages, etc. label Oct 26, 2022
Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

Nice job, thanks a lot!

I would want a 2nd review before merging.

I've made a nitpicking suggestion (which is not blocking the PR at all)

Copy link
Contributor

@kmartens27 kmartens27 left a comment

Choose a reason for hiding this comment

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

Hi @paulmarsicloud, thanks so much for your work on this tutorial! I made a few small suggestions on some formatting/syntax instances, but outside of that this looks great!
Let me know if you have any questions or feedback!

@kmartens27 kmartens27 added the hacktoberfest Hacktoberfest label Oct 26, 2022
@paulmarsicloud
Copy link
Contributor Author

Thank you @dduportal and @kmartens27 for the awesome feedback! I've gone ahead and applied those changes and re-pushed 👍

Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

Lgtm for me nice contribution!

Copy link
Member

@lemeurherve lemeurherve left a comment

Choose a reason for hiding this comment

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

Small nitpicks too, thanks for the PR!

@paulmarsicloud
Copy link
Contributor Author

Thanks for the feedback @lemeurherve - changes added 👍

Copy link
Contributor

@kmartens27 kmartens27 left a comment

Choose a reason for hiding this comment

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

Thanks for your work on this @paulmarsicloud, this is fantastic! We appreciate the contributions to help improve Jenkins!

@kmartens27 kmartens27 merged commit 7eddf41 into jenkins-infra:master Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Jenkins documentation, including user and developer docs, solution pages, etc. hacktoberfest Hacktoberfest hacktoberfest-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants