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: Fix warnings in Workflows and artifact name #17

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

qwersem
Copy link

@qwersem qwersem commented Feb 1, 2023

In this PR I fixed few warnings related with such deprecated tools as working OS, external actions and embedded github commands. Also, I noticed that the name of artifact doesn't have a version postfix which is resolved here.

Successful build of these changes you can see here: https://github.com/qwersem/arc-gnu-toolchain/actions/runs/4060081394

- Fix deprecated commands
- Update the action for checking out a repo
- Update host to toolchains build

Signed-off-by: Evgeny Semenov <[email protected]>
Signed-off-by: Evgeny Semenov <[email protected]>
@qwersem qwersem added the enhancement New feature or request label Feb 1, 2023
@qwersem qwersem requested a review from abrodkin February 1, 2023 08:30
@qwersem qwersem self-assigned this Feb 1, 2023
@qwersem qwersem linked an issue Feb 1, 2023 that may be closed by this pull request
Copy link
Member

@abrodkin abrodkin 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 cleaning this mess up. Now summary looks much nicer!
And indeed, now build date gets properly appended to the archive name.
@claziss OK to merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Fix warnings in Workflows
2 participants