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

add a script to print markdown list of submodule states #1256

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

tomeichlersmith
Copy link
Member

@tomeichlersmith tomeichlersmith commented Jan 29, 2024

This script mainly uses awk to look through the submodule states and
retrieve their versions. If a submodule is not on a clean version tag,
then we fall back to just linking to the commit its on.

Besides awk, this script also requires a version of git that has the
-C flag (>= 1.8.5, very easy on any newer systems, I've only run into issues
with this requirement on CentOS7 systems.) and all submodules to have
their remote named origin which is the git default so I expect only
super-users to see issues.

Use Case

This is helpful for anyone writing release notes. You can run this script and then copy the output into the release notes markdown to give an itemized list of the submodules linked to the commit/release that is included in that release.

In the future, we could look at including this script as part of a larger automatic release notes creation function. The current release notes generator just lists the PRs that were merged and then other notes are written manually. This script could be apart of a system that would itemize the list of PRs and the submodule versions and then leave a space for manually written notes at the top.

This script mainly uses awk to look through the submodule states and
retrieve their versions. If a submodule is not on a clean version tag,
then we fall back to just linking to the commit its on.

Besides awk, this script also requires a version of git that has the
`-C` flag (>= 1.8.5, very easy on any newer systems, I've only run into issues
with this requirement on CentOS7 systems.) and all submodules to have
their remote named `origin` which is the `git` default so I expect only
super-users to see issues.
@tomeichlersmith
Copy link
Member Author

Same as #1255 (comment)

@tomeichlersmith tomeichlersmith merged commit 7e51bb9 into trunk Jan 30, 2024
5 of 7 checks passed
@tomeichlersmith tomeichlersmith deleted the print-submodule-state-script branch January 30, 2024 19:22
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.

1 participant