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

Update github workflows #632

Merged
merged 1 commit into from
Feb 27, 2023

Conversation

mzuehlke
Copy link
Contributor

@mzuehlke mzuehlke commented Feb 27, 2023

runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: olafurpg/setup-scala@v13
- uses: actions/setup-java@3
Copy link
Contributor

@tgodzik tgodzik Feb 27, 2023

Choose a reason for hiding this comment

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

Suggested change
- uses: actions/setup-java@3
- uses: actions/setup-java@v3

I think 2 is the latest

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@tgodzik tgodzik Feb 27, 2023

Choose a reason for hiding this comment

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

actions/setup-java@v3 it looks like it needs the v then 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

🤦

@mzuehlke
Copy link
Contributor Author

I forgot to amend my last commit, will you squash them when merging ?

Copy link
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

You should be able to squash and force push with the changes

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
* olafurpg/setup got deprecated in v14
  * replaced by actions/setup-java@v3
  * enabled sbt caching
* olafurpg/setup-gpg is no longer required since sbt-ci-release v1.5.5
Copy link
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@tgodzik tgodzik merged commit 6af3033 into scalameta:main Feb 27, 2023
@mzuehlke mzuehlke deleted the github_workflow_maintenance branch February 27, 2023 15:57
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