Skip to content

SONARHTML-579 Update CODEOWNERS to Dynamic Languages Squad - #938

Merged
nathsou merged 1 commit into
masterfrom
nathan/update-dynamic-languages-codeowners
Nov 5, 2026
Merged

nathsou merged 1 commit into
masterfrom
nathan/update-dynamic-languages-codeowners

Conversation

@nathsou

@nathsou nathsou commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Update CODEOWNERS to the Dynamic Languages Squad following the Quality Domain repository ownership assignment.

-@SonarSource/code-quality-core-languages-parsers-squad
+@SonarSource/code-quality-dynamic-languages-squad

Evidence

  • Before: @SonarSource/code-quality-core-languages-parsers-squad owns .github/CODEOWNERS.
  • After: @SonarSource/code-quality-dynamic-languages-squad owns the same path. Verified the team has write access and the diff changes only this owner.
  • No application tests run; this is a CODEOWNERS-only change.

Merge Danger

Door: two-way

Revert the commit to restore the previous owner.

Blast Radius: reviews

Changes the team requested to review edits to .github/CODEOWNERS.

@nathsou
nathsou requested a review from a team November 5, 2026 12:56
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Update CODEOWNERS to Dynamic Languages Squad SONARHTML-579 Update CODEOWNERS to Dynamic Languages Squad Nov 5, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

SONARHTML-579

@gitar-bot

gitar-bot Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · Changing .github/CODEOWNERS redirects review requests to another squad, which could shift who reviews future ownership-file changes.

Updates .github/CODEOWNERS to assign ownership to the Dynamic Languages Squad, replacing the core languages parsers squad. No issues found.

Verification Rules 2/2 met PRs 4 inspected

Functional validation No results

Rules 2 passed 1 not applicable

✅ Internal artifact registries
✅ Repository license compliance

Broader impact PRs 4 inspected
Cross-PR 4 PRs inspected 0 open impacts

PR 8335: JS-2829 Update CODEOWNERS to Dynamic Languages Squad Inspected
PR 2319: Update CODEOWNERS to Dynamic Languages Squad Inspected
PR 9: Update CODEOWNERS to Dynamic Languages Squad Inspected
PR 8: Update CODEOWNERS to Dynamic Languages Squad Inspected

Auto-approval Not enabled

Set up

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqube-next

sonarqube-next Bot commented Nov 5, 2026

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@nathsou nathsou self-assigned this Nov 5, 2026
@nathsou
nathsou merged commit a022fe4 into master Nov 5, 2026
19 checks passed
@nathsou
nathsou deleted the nathan/update-dynamic-languages-codeowners branch November 5, 2026 13:09
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