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: create Go Generate workflow #72

Merged
merged 1 commit into from
Oct 17, 2022
Merged

Conversation

galargh
Copy link
Contributor

@galargh galargh commented Oct 17, 2022

Resolves #47

This implements a request described in #47 (comment)

We can also specify labels, assignees, reviewers and team-reviewers for the PR if we want.

Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

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

👌 I think this should be OK, there may be cases where the submodule will be updated but will not generate changes, but there shouldn't be many of those. Thanks @galargh!

@lidel lidel merged commit a2baec7 into multiformats:master Oct 17, 2022
@lidel
Copy link
Member

lidel commented Oct 17, 2022

@galargh just fyi test run https://github.com/multiformats/go-multicodec/actions/runs/3267378296 has this note:

The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Probably not something we need to fix asap, but it will impact all our actions, not just this one.

@galargh
Copy link
Contributor Author

galargh commented Oct 18, 2022

Yes, thanks for flagging. I've seen that this week. The good thing is they're unlikely to completely deprecate it any time soon since:

We are monitoring telemetry for the usage of these commands and plan to fully disable them in the future.

I created an issue for this in uCI - protocol/.github#405 - so that we get to it ahead of the February 2023 release. I'll do a sweep of all our orgs when I start working on that issue.

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.

Automate code generation via CI
3 participants