Skip to content

Conversation

uozopio
Copy link

@uozopio uozopio commented Jun 27, 2025

Summary

  • Removed the conflicting changelog.yml workflow to prevent duplicate releases
  • Kept release.yml as the single source of truth for the release process
  • This resolves race conditions where both workflows would trigger on push to main

Context

The repository had two competing workflows:

  1. changelog.yml: Used TriPSs/conventional-changelog-action to create changelogs and releases
  2. release.yml: Uses auto (by Intuit) for semantic versioning, changelog generation, and NPM publishing

Both workflows were triggering on push to main, causing:

  • Race conditions
  • Duplicate GitHub releases
  • Version conflicts
  • Inconsistent changelog formats

Solution

The release.yml workflow is more comprehensive and includes:

  • NPM publishing capabilities
  • Loop prevention with 'ci skip' mechanism
  • Already configured with .autorc file
  • Has @auto-it/first-time-contributor plugin installed

Therefore, changelog.yml has been removed to maintain a single, consistent release process.

Test plan

  • Verify changelog.yml is deleted
  • Confirm release.yml remains intact
  • Ensure .autorc configuration is preserved
  • Next push to main should trigger only one release workflow

Umutcan ÖNER added 10 commits June 27, 2025 14:21
- Add husky for git hooks management
- Add pre-commit hooks for formatting, linting, type checking, and security
- Add pre-push hooks for tests, build, and PR validation
- Add commitlint for conventional commit message validation
- Add lint-staged for efficient staged file processing
- Add validation scripts for branch naming, secrets detection, file sizes, and env files
- Enhance .gitignore patterns for environment files

This setup catches most CI/CD issues locally before they reach PRs, improving developer experience and reducing CI time.
- Remove deprecated husky v9 headers from hooks (preparing for v10)
- Fix lint-staged config to remove unsupported --write flag from ultracite format command
Copy link

❌ PR Validation Failed

Please ensure your PR meets all requirements:

  1. Branch Naming: Follow conventions like feat/description, fix/issue-123
  2. PR Title: Use format <type>(<scope>): <subject> (e.g., feat(auth): add OAuth support)
  3. PR Size: Keep changes under 5000 lines and 100 files
  4. Breaking Changes: Document with "!" in title and description

See our Contributing Guide for details.

Copy link

🤖 Reviewers have been automatically assigned based on the CODEOWNERS file. If you need different reviewers, please update manually.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Welcome to Zopio, @uozopio!

Thank you for your first pull request! We're thrilled to have you contribute to our community-driven project.

Next steps:

  • ✅ Ensure all CI checks pass
  • 📝 Fill out the PR template completely
  • 👀 A maintainer will review your PR soon
  • 💬 Feel free to ask questions if you need help

Helpful resources:

If your PR gets merged, you'll be added to our contributors list automatically!

Thank you for helping make Zopio better! 🚀 We appreciate your contribution!

@uozopio uozopio merged commit aafca27 into main Jun 27, 2025
9 of 12 checks passed
@uozopio uozopio deleted the feat/git-hooks-and-workflows branch June 27, 2025 12:13
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