Skip to content

Updates CodeOwners to be more fine-grained #3090

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Aug 16, 2025
Merged

Updates CodeOwners to be more fine-grained #3090

merged 7 commits into from
Aug 16, 2025
+36 −17

Conversation

pascal-roth
Copy link
Collaborator

Description

Updates CodeOwners to be more fine-grained

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have run the pre-commit checks with ./isaaclab.sh --format
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the changelog and the corresponding version in the extension's config/extension.toml file
  • I have added my name to the CONTRIBUTORS.md or my name already exists there

@pascal-roth pascal-roth self-assigned this Aug 5, 2025
@pascal-roth pascal-roth added the dev team Issue or pull request created by the dev team label Aug 5, 2025
jtigue-bdai
jtigue-bdai approved these changes Aug 5, 2025
View reviewed changes
Copy link
Collaborator

@jtigue-bdai jtigue-bdai left a comment

Choose a reason for hiding this comment

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

That looks good to me. As our team gets built back up I will reevaluate where I am code owner, for now this is fine.

kellyguo11
kellyguo11 reviewed Aug 6, 2025
View reviewed changes
Copy link
Contributor

@kellyguo11 kellyguo11 left a comment

Choose a reason for hiding this comment

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

Thanks a lot for putting this together!

ooctipus
ooctipus reviewed Aug 7, 2025
View reviewed changes
ooctipus
ooctipus reviewed Aug 7, 2025
View reviewed changes
@pascal-roth
Copy link
Collaborator Author

@Mayankm96 can you merge this one by chance, I still don't have the permission to merge PRs

@Mayankm96 Mayankm96 merged commit c445da8 into isaac-sim:main Aug 16, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hhansen-bdai hhansen-bdai hhansen-bdai approved these changes

@jtigue-bdai jtigue-bdai jtigue-bdai approved these changes

@ooctipus ooctipus ooctipus approved these changes

@kellyguo11 kellyguo11 Awaiting requested review from kellyguo11 kellyguo11 is a code owner

Assignees

@pascal-roth pascal-roth

Labels
dev team Issue or pull request created by the dev team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@pascal-roth @kellyguo11 @hhansen-bdai @jtigue-bdai @ooctipus @Mayankm96