-
Notifications
You must be signed in to change notification settings - Fork 5
updates for monorepo #85
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.
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
Open
grizz
wants to merge
12
commits into
main
Choose a base branch
from
grizz/2025-11-23
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Update version bump and tag * Add cli args * update cli args * change cli arg name * Use existing nogit arg
Migrate from Poetry to modern pyproject.toml with hatchling build backend to follow current Python packaging best practices. Replace legacy linting tools with ruff for better performance and unified tooling. Update CI to use uv for faster dependency resolution. Major changes: - Replace Poetry with hatchling build backend and uv dependency management - Replace black/isort/flake8/pyupgrade with comprehensive ruff configuration - Update CI workflows to use uv instead of poetry/tox - Fix deprecated pkg_resources import, use importlib.metadata instead - Remove Python 3.8 support (EOL) - Add py.typed marker file for type checking infrastructure - Apply ruff formatting to all source files - Upgrade all dependencies to latest compatible versions All 119 tests passing with 73% coverage.
The tox.ini file was still configured to run 'poetry install' which was causing CI failures since we migrated to uv. Updated to use 'uv sync --all-extras' and 'uv run pytest' instead. Fixes CI test failures on all Python versions.
The test_git_manager_sync_with_merge test performs git merges which create commits. Git requires user.name and user.email to be configured to create commits. Added git configuration step in the CI workflow to fix this test failure. Fixes: fatal: empty ident name (for <runner@...>) not allowed
Use the CODECOV_TOKEN secret to authenticate codecov uploads and avoid rate limiting issues.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #85 +/- ##
==========================================
- Coverage 77.23% 73.06% -4.17%
==========================================
Files 33 34 +1
Lines 2025 2599 +574
==========================================
+ Hits 1564 1899 +335
- Misses 461 700 +239 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Renamed the command-line flag from --nogit to --no-git-tag to make it clearer what the flag does (skips creating a git tag). The internal parameter name remains 'nogit' for backward compatibility. Updated documentation to reflect the new flag name.
Changed the internal variable name from 'nogit' to 'no_git_tag' for better clarity and consistency with the command-line flag. Updated files: - src/ctl/plugins/version_base.py - src/ctl/plugins/semver2.py - tests/util.py - tests/test_plugin_semver2.py All tests passing.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update version bump and tag
Add cli args
update cli args
change cli arg name
Use existing nogit arg