Skip to content

Conversation

@Paul-Ferrell
Copy link
Collaborator

Code review checklist:

  • Code is generally sensical and well commented
  • Variable/function names all telegraph their purpose and contents
  • Functions/classes have useful doc strings
  • Function arguments are typed
  • Added/modified unit tests to cover changes.
  • New features have documentation added to the docs.
  • New features and backwards compatibility breaks are noted in the RELEASE.md

@Paul-Ferrell Paul-Ferrell marked this pull request as draft April 13, 2025 04:40
@Paul-Ferrell Paul-Ferrell marked this pull request as ready for review April 22, 2025 20:38
@Paul-Ferrell Paul-Ferrell merged commit d270ae4 into develop Apr 30, 2025
8 checks passed
hwikle-lanl added a commit that referenced this pull request Jul 15, 2025
* Fixing issue with Python 5.11+ compatability.

- The argument parser is no longer a singleton.
- Directly included yapsy in Pavilion.

* OMG there were a lot of things to fix for python3.12.

* Added packaging dependency.

* Reverted packaging to a version that supports python 3.6.

* Fixed style issue.

* Fix find_all_tests method of resolver

It was finding old tests/* tests and suites/<name>/suite.yaml style tests,
but not suites/<name>.yaml style tests.

* Added the ability to specify suites by wildcards.

* Some fixes.

* Unit tests passing.

* Unit tests passing.

* Fixing style issues.

* Added unittest.

* Updated gitlab runner version.

* Updated python version.

* No github runners support python3.6 anymore.

* Fixed review issues.

---------

Co-authored-by: hwikle-lanl <154543628+hwikle-lanl@users.noreply.github.com>
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