Skip to content

Conversation

clintpurser
Copy link
Member

adds CODEOWNERS, test.yaml and dependabot.yaml

Copy link
Member

@jckras jckras left a comment

Choose a reason for hiding this comment

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

thanks for doing this on leave clint! left one nit, lgtm

@@ -0,0 +1,2 @@
* @clintpurser
* @viamrobotics/mobile No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

nit - we should add a space before the end of the file

Copy link
Member Author

Choose a reason for hiding this comment

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

i might be wrong, but i don't think it needs it. are you just saying that because of the warning?

Copy link
Member

Choose a reason for hiding this comment

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

Yes I am only saying it bc of the warning. it doesen't need it but I always think its nice to clear the warnings :) very much a "nit"

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