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

[try] Travis to github actions #1491

Closed
wants to merge 2 commits into from

Conversation

cesswairimu
Copy link
Collaborator

@cesswairimu cesswairimu commented Jan 31, 2021

Fixes #1489 (<=== Add issue number here)

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

  • PR is descriptively titled 📑 and links the original issue above 🔗
  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
  • code is in uniquely-named feature branch and has no merge conflicts 📁
  • screenshots/GIFs are attached 📎 in case of UI updation
  • ask @publiclab/mapknitter-reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

@codeclimate
Copy link

codeclimate bot commented Jan 31, 2021

Code Climate has analyzed commit 6fd441e and detected 0 issues on this pull request.

View more on Code Climate.

@katungi
Copy link
Contributor

katungi commented Jan 31, 2021

Hey @cesswairimu, I noticed the Travis build is being triggered instead of the new GitHub. Have you removed the travis config workflow files? That could possibly explain the travis workflow being triggered

with:
ruby-version: 2.4.6
bundler-cache: true
- uses: ./.github/actions/setup-test-environment
Copy link
Member

Choose a reason for hiding this comment

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

Aha - ok so this is missing this file, which we made in plots2 to reduce the length of the script: https://github.com/publiclab/plots2/blob/main/.github/actions/setup-test-environment/action.yml

Copy link
Member

Choose a reason for hiding this comment

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

Trying this in SWB here: publiclab/spectral-workbench#647

Copy link
Collaborator Author

@cesswairimu cesswairimu Jun 14, 2021

Choose a reason for hiding this comment

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

aha! thanks Jeff ...will look into it

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.

Move Travis to Github actions
3 participants