Skip to content

Add Nubots group and add them to flexible structures#273

Merged
pattacini merged 1 commit into
masterfrom
addNubotsFlexStructures
Apr 8, 2026
Merged

Add Nubots group and add them to flexible structures#273
pattacini merged 1 commit into
masterfrom
addNubotsFlexStructures

Conversation

@Nicogene
Copy link
Copy Markdown
Member

@Nicogene Nicogene commented Apr 8, 2026

As per title

@Nicogene Nicogene self-assigned this Apr 8, 2026
@Nicogene Nicogene requested a review from pattacini as a code owner April 8, 2026 09:12
@Nicogene Nicogene had a problem deploying to outside_collaborators_environment April 8, 2026 09:12 — with GitHub Actions Failure
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces new group definitions and repository permission configurations. It adds the nubots/everyone and nubots/mechanics groups and assigns write permissions for the nubots/mechanics group to the spark-study-flexible-structures repository. A review comment suggests removing the nubots/everyone group if it is not currently needed to keep the configuration clean.

Comment thread groups/nubots.yml
@pattacini
Copy link
Copy Markdown
Member

CI failed for an unrelated reason.
Fixed in master. Waiting to continue here.

@Nicogene Nicogene had a problem deploying to outside_collaborators_environment April 8, 2026 09:33 — with GitHub Actions Failure
@pattacini pattacini force-pushed the addNubotsFlexStructures branch from 85fe071 to 147c016 Compare April 8, 2026 09:47
@pattacini pattacini temporarily deployed to outside_collaborators_environment April 8, 2026 09:47 — with GitHub Actions Inactive
@pattacini pattacini merged commit 0a214c9 into master Apr 8, 2026
1 check passed
@pattacini pattacini deleted the addNubotsFlexStructures branch April 8, 2026 10:01
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