Skip to content

Add CODEOWNERS file - #2

Merged
dduugg merged 1 commit into
mainfrom
add-codeowners
Mar 5, 2026
Merged

Add CODEOWNERS file#2
dduugg merged 1 commit into
mainfrom
add-codeowners

Conversation

@dduugg

@dduugg dduugg commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Adds a .github/CODEOWNERS file assigning all files in the repository to @rubyatscale/ruby-at-scale.

@dduugg
dduugg marked this pull request as ready for review March 4, 2026 18:46
@dduugg
dduugg requested a review from a team March 5, 2026 17:51
@dduugg
dduugg merged commit c30321a into main Mar 5, 2026
3 checks passed
@dduugg
dduugg deleted the add-codeowners branch March 5, 2026 17:53
@dduugg
dduugg requested a review from a team March 5, 2026 17:53
dduugg added a commit that referenced this pull request Aug 17, 2026
Alert #1 (actions/missing-workflow-permissions,
.github/workflows/main.yml:12) — the single `build` job only checks out
the repo, sets up Ruby, and runs `bundle exec rake` (rspec + rubocop).
No writes of any kind, so a workflow-level `contents: read` is the
correct least privilege. Placed at the workflow level after `on:`, which
matches the existing style in this repo's release.yml.

Alert #2 (rb/shell-command-constructed-from-input,
lib/chatwerk/cli.rb:20) — the `inspect` Thor command interpolated the
user-supplied working directory into a single string passed to `system`,
so Ruby handed it to /bin/sh. Passing separate argv entries bypasses the
shell entirely; the `PWD=#{pwd}` interpolation is now a single argv
element and can no longer break out of its argument. No spec stubs or
asserts on this call, and the observable command is unchanged.
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