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

ci: use GitHub CLI to download M+ libmicroros #271

Merged
merged 3 commits into from
Jun 27, 2024
Merged

Conversation

gavanderhoorn
Copy link
Collaborator

As per title.

This switches to the GH CLI for interaction with releases of micro_ros_motoplus from the currently used dsaltares/fetch-gh-release-asset.

This works just as well and also supports downloading assets from draft releases (which we'll need later).

Instead of dsaltares/fetch-gh-release-asset.

Works just as well and also supports downloading assets from draft
releases (which we'll need later).
@gavanderhoorn gavanderhoorn requested a review from ted-miller June 27, 2024 10:35
@gavanderhoorn
Copy link
Collaborator Author

gavanderhoorn commented Jun 27, 2024

Unfortunately the (misconfigured) msbuild_filter job prevented the msbuild job from running, but a build with this can be found here. Be sure to choose one of the builds and expand the Download M+ libmicroros (...) step.

(note: that build failed, but not because of the changes in this PR, just the known issues with the 20221102 release of M+ libmicroros)

GH CLI requires a token to download releases when used in GHA, so
acquire one and pass it to GH CLI.
That way we don't have to start looking for it after the download is complete, but can just assume it has a certain name.
@ted-miller ted-miller merged commit fd51200 into main Jun 27, 2024
7 of 16 checks passed
@ted-miller ted-miller deleted the ci_switch_to_gh_cli branch June 27, 2024 18:03
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