Skip to content

Conversation

zachmprince
Copy link
Contributor

Refs #31668

@zachmprince zachmprince force-pushed the conda_moose_python branch 2 times, most recently from 76f592d to 419db7d Compare October 6, 2025 13:59
@moosebuild
Copy link
Contributor

Job Precheck, step Versioner verify on 419db7d wanted to post the following:

Versioner templates

Found 17 templates, 0 failed

Versioner influential files

Found 69 influential files, 0 changed, 11 added, 0 removed

package status file
python NEW conda/python/conda_build_config.yaml
python NEW conda/python/meta.yaml
python NEW modules/geochemistry/python
python NEW modules/level_set/python
python NEW modules/navier_stokes/python
python NEW modules/stochastic_tools/python
python NEW modules/thermal_hydraulics/python/moose_thm/init.py
python NEW modules/thermal_hydraulics/python/moose_thm/thm_utilities.py
python NEW python/MooseControl
python NEW python/mms
python NEW python/mooseutils

Versioner versions

Found 10 packages, 1 changed, 0 failed

package status hash to hash version to version
python NEW none 39b011a none 2025.10.02

@moosebuild
Copy link
Contributor

Job Documentation, step Docs: sync website on 419db7d wanted to post the following:

View the site here

This comment will be updated on new commits.

self.assertEqual(Versioner.match_date('xxx2025.04.04xxx'), (2025, 4, 4))
self.assertEqual(Versioner.match_date('20.01.01'), None)

class TestVerify(unittest.TestCase):
Copy link
Member

Choose a reason for hiding this comment

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

Can you please add docstrings for the tests that briefly describe what they're doing and how

continue
change_text = 'BEHIND' if changes else colorize('OK', 'GREEN')
entries.append((package.name, change_text, to_path, template_path))
entries.append([package.name, change_text, to_path, template_path])
Copy link
Member

Choose a reason for hiding this comment

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

Why tuple to list?

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