Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Carroll/branching task #1491

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Carroll/branching task #1491

wants to merge 8 commits into from

Conversation

BC-UON
Copy link

@BC-UON BC-UON commented Oct 5, 2024

Things the PR does go here

  • these can be bullet
  • points like this
  • if you like

Pre-PR Checklist:

Have you

  • Updated NUbook if necessary (add link to NUbook PR here)
  • Added/updated tests for your changes, including regression tests for bug fixes
  • Updated relevant module READMEs
  • Added/modified documentation directives in relevant code
  • Added a descriptive title and relevant labels to the PR

Optional Headings (delete this heading and the ones you don't use)

Reviewers, Note

Things the reviewers should keep in mind go here.
Also tell reviewers how to verify/test your code's correctness here.

Things left to do

  • If you have more things to do
  • Add them here

If there are many things left to do then consider a draft PR

Labels Info (delete the rest of this once you have read it)

We use GitHub's Labels to categorise PRs and Issues. Most PRs should have these labels:

  • G-Group-Name: G for the group(s) associated with or reviewing the PR.
  • L-Language-Name: L for the language(s) which are used most in the PR.

Less common label categories include:

  • !!-High-Priority-Reason: A reason this PR is high priority e.g. it's a bug or it's explicitly for RoboCup.
  • -General-Thing: A general PR thing. These are all case-by-case and don't fit well into the other categories.
  • -D-Difficulty-Level: The difficulty level of implementing the change. This is mostly used for issues and is usually not applicable to PRs.

A full list of labels is available here.

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.

3 participants