Skip to content

Conversation

@vilsonrodrigues
Copy link
Contributor

Summary

Fixes two issues with the publish workflow:

  1. Version extraction failure: The workflow was trying to import msgspec_ext.version before installing dependencies, causing ModuleNotFoundError: No module named 'msgspec'. Changed to use grep to extract version directly from version.py.

  2. Labeler workflow failures: Disabled the labeler workflow as it consistently fails on fork PRs due to GITHUB_TOKEN permission limitations.

Changes

  • Use grep instead of Python import to extract version (both current and in create-release job)
  • Add if: false to labeler workflow to disable it
  • Add explanatory comments

Testing

The publish workflow should now complete successfully when triggered by a release PR merge.

Related

vilsonrodrigues and others added 7 commits November 27, 2025 02:18
- Complete project structure overview
- Common commands and workflows
- Release process (always use ./scripts/release.sh)
- Architecture details and optimizations
- Linting, testing, and CI/CD guides
- Troubleshooting tips

This file provides context for Claude Code to work more effectively
with the project without repeating instructions.
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
The publish workflow was failing because it tried to import msgspec_ext
before installing dependencies. The __init__.py imports settings.py which
requires msgspec, causing ModuleNotFoundError.

Changed to use grep to extract version directly from version.py file,
similar to how we already extract the previous version.
The labeler workflow fails on fork PRs due to GITHUB_TOKEN permission
limitations. Added 'if: false' to disable it while keeping the
configuration for potential future use.
@vilsonrodrigues vilsonrodrigues merged commit b52ec5b into msgflux:main Dec 2, 2025
10 checks 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.

1 participant