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

Add wl_ugps_external #73

Merged

Conversation

clydemcqueen
Copy link
Contributor

Hi, folks -- I'm submitting my super-simple "Water Linked UGPS External" extension for the extensions repository. Thanks.

Copy link
Member

@Williangalvani Williangalvani left a comment

Choose a reason for hiding this comment

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

Nice!

@Williangalvani
Copy link
Member

@clydemcqueen I just noticed you don't have any tags in your github repo, wee need those for loading the readme as you are doing https://github.com/clydemcqueen/wl_ugps_external_extension/blob/main/Dockerfile#L59

@clydemcqueen
Copy link
Contributor Author

@Williangalvani Thanks for the help. I think I fixed it -- can you take another look?

@Williangalvani
Copy link
Member

perfect!

@Williangalvani Williangalvani merged commit 2e7cac7 into bluerobotics:master Nov 11, 2023
2 checks passed
@clydemcqueen clydemcqueen deleted the clyde_wl_ugps_external branch November 11, 2023 19:33
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