Skip to content

Conversation

@mshafer-NI
Copy link
Collaborator

The list of steps in the example for analyze-project skips repo checkout (which is required).

Updating this list makes it easier for users to just copy/paste the example usage.

What does this Pull Request accomplish?

Make documentation clearer

Why should this Pull Request be merged?

It cost me 20 minutes of failed builds to realize that the repo just wanted synced yet...

What testing has been done?

Used the corrected list in a PR in ni-python-styleguide

The list of steps in the example for analyze-project skips repo checkout (which is required).

Updating this list makes it easier for users to just copy/paste the example usage.
@mshafer-NI mshafer-NI requested a review from bkeryan as a code owner November 14, 2025 21:50
@mshafer-NI mshafer-NI merged commit 227a13d into main Nov 14, 2025
195 checks passed
@mshafer-NI mshafer-NI deleted the users/mshafer-NI/update-anlyze-readme branch November 14, 2025 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants