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 checks in GitHub actions #2813

Merged
merged 12 commits into from
Feb 2, 2022
Merged

CI checks in GitHub actions #2813

merged 12 commits into from
Feb 2, 2022

Conversation

ennru
Copy link
Member

@ennru ennru commented Jan 29, 2022

Moves some checks from Travis to GitHub actions
Refs lagom/lagom#3331

@ennru ennru force-pushed the ennru-towards-actions branch from d97ddc4 to cf596b4 Compare January 29, 2022 17:45
Comment on lines +12 to +15
concurrency:
# Only run once for latest commit per ref and cancel other (previous) runs.
group: ci-${{ github.ref }}
cancel-in-progress: true
Copy link
Member

Choose a reason for hiding this comment

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

Hmm, interesting. I will add it to Lagom as well.

@ennru ennru force-pushed the ennru-towards-actions branch from dbd1e5a to 2d437fb Compare February 1, 2022 10:36
@ennru ennru removed the p:ftp label Feb 1, 2022
@ennru
Copy link
Member Author

ennru commented Feb 1, 2022

The FTP integration tests don't work and require further investigation.
See even #1810

@ennru ennru marked this pull request as ready for review February 1, 2022 10:43
Copy link
Member

@octonato octonato left a comment

Choose a reason for hiding this comment

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

LGTM

@octonato
Copy link
Member

octonato commented Feb 2, 2022

I relaunched this build to verify an issue we are facing in a Lagom build (lagom/lagom#3336). And it's also failing here.

The coursier/setup-action@v1 is failing to download sbtn.

@ennru
Copy link
Member Author

ennru commented Feb 2, 2022

This setup-action error is induced by coursier/setup-action#171 and coursier/setup-action#163

@ennru ennru merged commit afe6054 into akka:master Feb 2, 2022
@ennru ennru deleted the ennru-towards-actions branch February 2, 2022 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants