Skip to content

S fairchild/aro 25110#4676

Closed
s-fairchild wants to merge 2 commits intomasterfrom
s-fairchild/ARO-25110
Closed

S fairchild/aro 25110#4676
s-fairchild wants to merge 2 commits intomasterfrom
s-fairchild/ARO-25110

Conversation

@s-fairchild
Copy link
Copy Markdown
Collaborator

Which issue this PR addresses:

https://issues.redhat.com/browse/ARO-25110

What this PR does / why we need it:

Updates and consolidates our development environment setup documentation.

Test plan for issue:

I've locally tested the bash helpers in my Fedora development environment.

Is there any documentation that needs to be updated for this PR?

Docs are in this PR.

How do you know this will function as expected in production?

N/A development environment doc updates.

Add optional information on how to install Go via gvm
Add variable to reference golint version to be installed
  * This prevents us from hardcoding a version in the setup doc.
Add makefile.md documentation
  * Seperating this simplifies our primary dev-env-setup document.
Add bash environment helpers documentation
Update dev-setup docs relocating Debian and Mac OS dependencies
  * They should be grouped under all OS specific dependencies to be installed.
Fix formatting issues
…local rp development

The primary setup document was getting very cluttered. This helps it to be more easily read and understood.
@s-fairchild s-fairchild added documentation Improvements or additions to documentation loki labels Mar 10, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 10, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Comment on lines +2 to +9
- [Bash Environment Helpers](#bash-environment-helpers)
- [ARO-RP Bash Environment Helpers](#aro-rp-bash-environment-helpers)
- [Helper Aliases and Functions](#helper-aliases-and-functions)
- [Aliases](#aliases)
- [Functions](#functions)
- [Bash rc config Drop-Ins](#bash-rc-config-drop-ins)
- [bashrc.d Setup](#bashrcd-setup)
- [bashrc Drop-Ins Testing](#bashrc-drop-ins-testing)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I wanted to note that GitHub auto-generates a TOC for rendered markdown files.

Image

Then we don't have to maintain a TOC, or forget to update it which results in dead anchor links.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is true.
I personally render and read markdown docs locally in vscode... so I appreciate having the TOC still present.

@s-fairchild
Copy link
Copy Markdown
Collaborator Author

Changes are included in #4680

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation loki

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants