Skip to content

fix: Fix versioning#137

Merged
chichun-charlie-liu merged 4 commits into
foundation-model-stack:mainfrom
tharapalanivel:fix_ver
Jun 10, 2025
Merged

fix: Fix versioning#137
chichun-charlie-liu merged 4 commits into
foundation-model-stack:mainfrom
tharapalanivel:fix_ver

Conversation

@tharapalanivel

@tharapalanivel tharapalanivel commented Jun 10, 2025

Copy link
Copy Markdown
Collaborator

Description of the change

This PR allows us to get the version for the fms_mo package correctly, previously it was falling back to 0.0.0

Related issues or PRs

How to verify the PR

Install package

pip install -e .

and

>>> import fms_mo
>>> fms_mo.__version__
'0.1.1.dev360'

Was the PR tested

  • I have added >=1 unit test(s) for every new method I have added (if that coverage is difficult, please briefly explain the reason)
  • I have ensured all unit tests pass

Checklist for passing CI/CD:

  • All commits are signed showing "Signed-off-by: Name <email@domain.com>" with git commit -signoff or equivalent
  • PR title and commit messages adhere to Conventional Commits
  • Contribution is formatted with tox -e fix
  • Contribution passes linting with tox -e lint
  • Contribution passes spellcheck with tox -e spellcheck
  • Contribution passes all unit tests with tox -e unit

Note: CI/CD performs unit tests on multiple versions of Python from a fresh install. There may be differences with your local environment and the test environment.

Signed-off-by: Thara Palanivel <130496890+tharapalanivel@users.noreply.github.com>
Signed-off-by: Thara Palanivel <130496890+tharapalanivel@users.noreply.github.com>
Signed-off-by: Thara Palanivel <130496890+tharapalanivel@users.noreply.github.com>
@tharapalanivel tharapalanivel changed the title Fix versioning fix: Fix versioning Jun 10, 2025
@github-actions github-actions Bot added the fix label Jun 10, 2025
Signed-off-by: Thara Palanivel <130496890+tharapalanivel@users.noreply.github.com>
@chichun-charlie-liu chichun-charlie-liu merged commit 7467f68 into foundation-model-stack:main Jun 10, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants