Skip to content

Add workflow labels and fix test workflow#11

Merged
kompre merged 7 commits into
mainfrom
dev
Oct 21, 2025
Merged

Add workflow labels and fix test workflow#11
kompre merged 7 commits into
mainfrom
dev

Conversation

@kompre
Copy link
Copy Markdown
Owner

@kompre kompre commented Oct 20, 2025

Summary

Merging changes from dev to main:

  • Created repository labels for workflow automation
  • Fixed YAML syntax error in test workflow

Changes

Labels Created (via #10)

Created 12 repository labels via GitHub API:

Workflow Triggers:

  • code-review - Trigger automated Claude code reviews
  • release - Publish package to PyPI
  • test-release - Publish package to TestPyPI

Version Bumps:

  • bump:major, bump:minor, bump:patch - Semantic versioning
  • bump:stable, bump:alpha, bump:beta, bump:rc, bump:post, bump:dev - Pre-release versions

YAML Syntax Fix

Fixed malformed YAML in .github/workflows/claude-code-review.yml:

  • Removed incorrect nested job structure
  • Corrected job name to code-review

Documentation

  • Added CLAUDE.md with project guidance
  • Added _todo/ workflow system for task management
  • Documented repository labels in CLAUDE.md

Testing

✅ All 17 tests pass

Merged PRs

🤖 Generated with Claude Code

kompre and others added 4 commits October 20, 2025 21:33
* Add repository labels for workflows

Created all necessary labels for code-review and release workflows:
- code-review: Trigger automated Claude code reviews
- release/test-release: PyPI publishing workflow triggers
- bump:*: Version bump labels (major, minor, patch, stable, alpha, beta, rc, post, dev)

Updated CLAUDE.md with label documentation.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Fix test workflow YAML syntax error

Fixed malformed YAML in claude-code-review.yml that was causing workflow
failures. Changed job name from incorrect nested structure to proper
'code-review' job name.

Verified all 17 tests pass locally (16.12s).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
Changed Quarto version from 'latest' to specific version '1.8.25' to avoid
bug in quarto-dev/quarto-actions/setup@v2 that constructs malformed URL
when using 'latest'.

Error was: wget https://github.com/.../vlatest/quarto-latest-linux-amd64.deb
(404 Not Found)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Implement label-driven release automation with version bumping:

- Add version-bump.yml workflow
  - Triggers on PR labels (bump:major, bump:minor, etc.)
  - Applies version bumps using uv version --bump
  - Commits changes back to PR branch
  - Posts version change comment

- Add release.yml workflow
  - Triggers on merged PRs with release/test-release labels
  - Builds and tests package
  - Publishes to PyPI or TestPyPI
  - Creates git tag and GitHub Release

- Remove deprecated python-publish.yml workflow

- Add comprehensive release documentation (.github/RELEASE.md)

- Update CLAUDE.md with release workflow guidance

All required labels already exist in repository. Ready for testing phase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@kompre kompre added test-release Publish package to TestPyPI bump:minor Version bump: minor bump:rc Version bump: rc labels Oct 21, 2025
Issues fixed:
- Remove 'synchronize' trigger to prevent multiple workflow runs
- Apply all version bumps in single uv command instead of sequential
- Now correctly handles: uv version --bump minor --bump rc

This fixes PR #11 where adding multiple labels triggered 3 separate
workflow runs, each failing because they tried to bump separately.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@kompre kompre added test-release Publish package to TestPyPI bump:minor Version bump: minor bump:rc Version bump: rc and removed test-release Publish package to TestPyPI bump:minor Version bump: minor bump:rc Version bump: rc labels Oct 21, 2025
@github-actions
Copy link
Copy Markdown
Contributor

✅ Version bumped: 2025.8.112025.9.0rc1

@kompre kompre closed this Oct 21, 2025
@kompre kompre reopened this Oct 21, 2025
@kompre kompre merged commit 8da098a into main Oct 21, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump:minor Version bump: minor bump:rc Version bump: rc test-release Publish package to TestPyPI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant