Skip to content

fix: version comment and permissions#8

Merged
jccguimaraes merged 2 commits intomainfrom
fix/version-comment-and-permissions
Apr 10, 2026
Merged

fix: version comment and permissions#8
jccguimaraes merged 2 commits intomainfrom
fix/version-comment-and-permissions

Conversation

@jccguimaraes
Copy link
Copy Markdown
Contributor

No description provided.

… comment

Replaces the upsertUsesComment approach (which inserted a // comment
above the uses block) with an inline # comment on the version line,
e.g. version = "abc123..." # v4. The # style aligns with HCL conventions
and mirrors how GitHub Actions YAML documents pinned SHAs.
…permissions block

An empty permissions {} block now outputs permissions: {} (all scopes
denied) rather than permissions: read-all, following least-privilege
best practice for GitHub Actions.
@github-actions
Copy link
Copy Markdown

Go test coverage: 64.9% for commit 5ec599e
⚠️ 1 of 19 packages have zero coverage.
  • github.com/yldio/cinzel/internal/test
View coverage for all packages
# Package Name                                     | Coverage
+ github.com/yldio/cinzel                          |    80.0%
+ github.com/yldio/cinzel/internal/ai              |    46.8%
+ github.com/yldio/cinzel/internal/cinzelerror     |    18.2%
+ github.com/yldio/cinzel/internal/command         |    39.5%
+ github.com/yldio/cinzel/internal/filereader      |    50.0%
+ github.com/yldio/cinzel/internal/filewriter      |    88.9%
+ github.com/yldio/cinzel/internal/fsutil          |    68.0%
+ github.com/yldio/cinzel/internal/hclparser       |    62.7%
+ github.com/yldio/cinzel/internal/maputil         |    93.3%
+ github.com/yldio/cinzel/internal/naming          |    93.1%
+ github.com/yldio/cinzel/internal/pin             |    54.8%
- github.com/yldio/cinzel/internal/test            |     0.0%
+ github.com/yldio/cinzel/internal/yamlwriter      |    88.9%
+ github.com/yldio/cinzel/provider/github          |    69.4%
+ github.com/yldio/cinzel/provider/github/action   |    82.1%
+ github.com/yldio/cinzel/provider/github/job      |    92.6%
+ github.com/yldio/cinzel/provider/github/step     |    69.3%
+ github.com/yldio/cinzel/provider/github/workflow |    85.6%
+ github.com/yldio/cinzel/provider/gitlab          |    63.5%

@jccguimaraes jccguimaraes merged commit 41c8925 into main Apr 10, 2026
5 checks passed
@jccguimaraes jccguimaraes deleted the fix/version-comment-and-permissions branch April 10, 2026 13:31
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