Skip to content

Update bug report and pull request templates #888

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.

Sign up for GitHub

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

Jump to bottom
Open
tjlaboss wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Update bug report and pull request templates #888

tjlaboss wants to merge 2 commits into from

Conversation

@tjlaboss
Copy link
Collaborator

@tjlaboss tjlaboss commented Jan 22, 2026 •
edited by github-actions bot
Loading

Pull Request Checklist for MontePy

Description

  • Update bug report template's example to be valid MCNP
  • Update black version in PR template to allow version 26

Fixes #637 and #886

LLM Disclosure

Were any large language models (LLM or "AI") used in to generate any of this code?

  • Yes
    • Model(s) used:
  • No

Documentation Checklist
  • I have documented all added classes and methods.
  • For infrastructure updates, I have updated the developer's guide.
  • For significant new features, I have added a section to the getting started guide.

Additional Notes for Reviewers

Ensure that:

  • This PR fully addresses and resolves the referenced issue(s).
  • The submitted code is consistent with the merge checklist outlined here.
  • The PR covers all relevant aspects according to the development guidelines.
  • 100% coverage of the patch is achieved, or justification for a variance is given.

📚 Documentation preview 📚: https://montepy--888.org.readthedocs.build/en/888/

@tjlaboss tjlaboss requested a review from MicahGale January 22, 2026 19:27
@tjlaboss
Copy link
Collaborator Author

tjlaboss commented Jan 22, 2026

🎱🎱🎱

MicahGale
MicahGale approved these changes Jan 22, 2026
View reviewed changes
Copy link
Collaborator

@MicahGale MicahGale left a comment

Choose a reason for hiding this comment

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

LGTM.

Do you want to piggy back #886 into this PR?

@tjlaboss
Copy link
Collaborator Author

tjlaboss commented Jan 22, 2026

yeah

@tjlaboss tjlaboss changed the title Update bug report template to be valid MCNP Update bug report and pull request templates Jan 22, 2026
@tjlaboss tjlaboss requested a review from MicahGale January 22, 2026 21:37
MicahGale
MicahGale approved these changes Jan 22, 2026
View reviewed changes
Copy link
Collaborator

@MicahGale MicahGale left a comment

Choose a reason for hiding this comment

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

LGTM.

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

Reviewers

@MicahGale MicahGale MicahGale approved these changes

Assignees

No one assigned

Labels

low priority

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bug Report Template: Make example file valid

3 participants

@tjlaboss @MicahGale