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.

Sign up for GitHub

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

Jump to bottom

chore(ci): converted travis workflow to github action #718

Merged
merged 5 commits into from
Apr 20, 2021
Merged

chore(ci): converted travis workflow to github action #718

merged 5 commits into from
Apr 20, 2021

Conversation

marcuslyons
Copy link
Contributor

@marcuslyons marcuslyons commented Apr 19, 2021 •
edited
Loading

vweevers
vweevers requested changes Apr 19, 2021
View reviewed changes
Copy link
Member

@vweevers vweevers left a comment

Choose a reason for hiding this comment

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

Thank you so much and welcome! Needs two small changes.

.github/workflows/test.yml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@vweevers vweevers mentioned this pull request Apr 19, 2021
Closed
60 tasks
marcuslyons and others added 2 commits April 19, 2021 13:31
@marcuslyons @vweevers
Update .github/workflows/sauce.yml
350ae84 
Co-authored-by: Vincent Weevers <mail@vincentweevers.nl>
@marcuslyons
chore(ci): remove action conditional;
dc01d1c
vweevers
vweevers approved these changes Apr 19, 2021
View reviewed changes
Copy link
Member

@vweevers vweevers left a comment

Choose a reason for hiding this comment

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

I'll add the necessary secrets tomorrow and push to a branch (that has access to the secrets) so that we can see it in action. Nice work!

@mxttwoods mxttwoods mentioned this pull request Apr 19, 2021
Closed
@vweevers vweevers merged commit e5453d7 into Level:master Apr 20, 2021
@vweevers
Copy link
Member

Follow-ups: 521b34c and f4d6a1d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ralphtheninja ralphtheninja ralphtheninja left review comments

@vweevers vweevers vweevers approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@marcuslyons @vweevers @ralphtheninja