Skip to content

Conversation

@raulfdm
Copy link
Owner

@raulfdm raulfdm commented Apr 7, 2025

This pull request introduces several enhancements and fixes to the @codeowners-flow/cli package, including a new command, improved string formatting, and refactoring for better code organization and readability. The most important changes are grouped by theme below.

New Features:

  • Added a new "validate" command to validate the CODEOWNERS file against a set of rules (.changeset/light-keys-crash.md, packages/cli/src/index.ts, packages/cli/src/validate-codeowners.ts, packages/cli/src/validate-codeowners.test.ts). [1] [2] [3] [4] [5]

Enhancements:

  • Enhanced string formatting while generating CODEOWNERS (.changeset/chilly-clowns-start.md).
  • Fixed init boilerplate to import defineConfig from CLI (.changeset/purple-candies-matter.md, packages/cli/src/init.ts, packages/cli/src/init.test.ts). [1] [2] [3] [4] [5]

Refactoring:

  • Refactored generateCodeOwners function to separate concerns and improve readability (packages/cli/src/generate-codeowners.ts).
  • Updated getCodeOwnersContent to use dedent for better formatting (packages/cli/src/get-codeowners-content.ts). [1] [2]

Dependency Updates:

  • Added dedent dependency to package.json (packages/cli/package.json).

Codeowners Configuration:

  • Simplified CODEOWNERS file by consolidating team assignments (.github/CODEOWNERS, packages/cli/src/__fixtures__/codeowners.config.mjs). [1] [2]

@changeset-bot
Copy link

changeset-bot bot commented Apr 7, 2025

🦋 Changeset detected

Latest commit: ed79437

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@codeowners-flow/cli Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@raulfdm raulfdm merged commit 819e1aa into main Apr 7, 2025
3 checks passed
@raulfdm raulfdm deleted the add-validator branch April 7, 2025 05:07
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