Skip to content

Add PR templates for adding and rebuilding software#1474

Open
ocaisa wants to merge 15 commits intoEESSI:mainfrom
ocaisa:main
Open

Add PR templates for adding and rebuilding software#1474
ocaisa wants to merge 15 commits intoEESSI:mainfrom
ocaisa:main

Conversation

@ocaisa
Copy link
Copy Markdown
Member

@ocaisa ocaisa commented Apr 17, 2026

This works in my testing on my own repo.

@@ -0,0 +1 @@
PULL_REQUEST_TEMPLATE/add_software.md No newline at end of file
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This creates a default template, without this you need to manually specify the template to use in your URL, like https://github.com/ocaisa/software-layer/compare/main...ocaisa:software-layer:dummy?template=rebuild_software.md

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

and ending with what is at the end of this one, so they can be multiline!) -->

(If you need a PR template for a rebuild, open the Preview tab in this comment
and [click here](?expand=1&template=rebuild_software.md). Otherwise delete this line)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is a workaround for the fact that a PR template picker is not generally available (approach taken from https://github.com/orgs/community/discussions/4620#discussioncomment-3383383)


<!-- Mark relevant options with an [x] -->
- [ ] I am targeting the **correct EESSI version** (e.g. `2025.06` for toolchain families `2025a`, `2023.06` for toolchain families `2023b`)
- [ ] I am using the **latest EasyBuild version** (e.g. `5.3.0`)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please verify:

<!-- Mark relevant options with an [x] -->
- [ ] I am targeting the **correct EESSI version** (e.g. `2025.06` for toolchain families `2025a`, `2023.06` for toolchain families `2023b`)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

<!-- Mark relevant options with an [x] -->
- [ ] I am targeting the **correct EESSI version** (e.g. `2025.06` for toolchain families `2025a`, `2023.06` for toolchain families `2023b`)
- [ ] I am using the **latest EasyBuild version** (e.g. `5.3.0`)
- [ ] I selected the **correct toolchain family** (e.g. `2025a`)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

family -> generation ?

- [ ] I am targeting the **correct EESSI version** (e.g. `2025.06` for toolchain families `2025a`, `2023.06` for toolchain families `2023b`)
- [ ] I am using the **latest EasyBuild version** (e.g. `5.3.0`)
- [ ] I selected the **correct toolchain family** (e.g. `2025a`)
- [ ] I did **not** modify older or unrelated files
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

older or other?


## Testing
<!-- Describe how you validated your changes -->
- [ ] I have tested this PR locally using **EESSI-extend**
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

add link to docs

-->
- [ ] The build completed successfully
- [ ] The installed software/module loads correctly
- [ ] Basic functionality has been verified
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

give example, like example --help ?

Copy link
Copy Markdown
Member Author

@ocaisa ocaisa Apr 17, 2026

Choose a reason for hiding this comment

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

I don't want to make this too big, what this might mean is very software specific so I don't have a problem with being a little vague (ideally the easyconfig is doing such basic checks anyway)

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.

2 participants