Skip to content

fix(ci): stabilize main validation and release workflow - #9

Merged
FireHead90544 merged 2 commits into
FireHead90544:mainfrom
ashishsinghbora:fix/stabilize-main-release-ci
Oct 1, 2026
Merged

FireHead90544 merged 2 commits into
FireHead90544:mainfrom
ashishsinghbora:fix/stabilize-main-release-ci

Conversation

@ashishsinghbora

@ashishsinghbora ashishsinghbora commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

I fixed the reliability issues around main, package validation, and the release workflow.

The main goal was to make the project easier to verify before merge and safer to release without changing the core how-cli functionality.

Changes

  • Added automated CI for pull requests and main
  • Added CLI smoke checks
  • Added package build validation
  • Added twine check before release upload
  • Added clean-install verification for built distributions
  • Hardened the release workflow for future versions
  • Preserved the existing provider and CLI behavior
  • Kept the changes focused without unrelated refactoring

Validation

  • Pytest: 60 passed locally and in GitHub Actions
  • Package build: passed (python -m build)
  • Twine validation: passed for wheel and sdist
  • Clean wheel install: passed in a temporary Python 3.14 environment; import resolved from site-packages
  • how --version: passed (v0.3.0)
  • how --help: passed
  • python -m how --help: passed
  • Diff check: passed (git diff --check)
  • GitHub Actions: passed (CI run 36507345805)

Notes

0.3.0 is already published on PyPI, so this PR does not attempt to republish that version. The release improvements are for future tagged versions.

The core application behavior is unchanged.

@ashishsinghbora

Copy link
Copy Markdown
Contributor Author

Done — stabilized main and release flow

I went through the current main flow and fixed the CI/release reliability problems instead of just patching the visible error.

What I changed:

  • Added proper PR/main validation
  • Added CLI smoke tests
  • Added package build + distribution validation
  • Added clean wheel installation testing
  • Hardened the PyPI release workflow
  • Added checks before future releases are uploaded
  • Kept the existing CLI/provider functionality intact

I also verified the final implementation locally and checked the GitHub Actions configuration.

Validation

  • Pytest: 60 passed locally and in GitHub Actions
  • Build: passed (python -m build)
  • twine check: passed for wheel and sdist
  • Clean install: passed; import came from temporary-environment site-packages
  • CLI smoke tests: passed (how --version, how --help, python -m how --help, how setup --help, how to --help, how fix --help)
  • Git diff check: passed
  • CI: passed on GitHub Actions (run 36507345805)

The changes are pushed in this PR and the branch is ready for review.

— Ashish

@FireHead90544

Copy link
Copy Markdown
Owner

Sweet, that's some significant non-functional improvements. Looks good, I'm out of my workspace so I can't really test them out, but since all the validation tests are passing, I'm assuming it's all good. So I'll merge it.

Before I merge, could you please bump the patch version from 0.3.0 to 0.3.1. The merge will then auto build and update the distribution on PyPI.

@ashishsinghbora

@ashishsinghbora

ashishsinghbora commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor Author

@FireHead90544 Updated PR #9 with the requested patch version bump from 0.3.0 to 0.3.1.

Revalidated the package after the change:

  • pytest passed
  • package build passed
  • twine check passed
  • CLI version reports 0.3.1

No unrelated changes were included. Ready for merge.

@FireHead90544 FireHead90544 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Perfect, merging.

@FireHead90544
FireHead90544 merged commit 07b77e9 into FireHead90544:main Oct 1, 2026
1 check passed
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.

2 participants