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

Replace Travis with GitHub Actions #1334

Merged
merged 3 commits into from
Dec 27, 2020
Merged

Conversation

ogonkov
Copy link
Contributor

@ogonkov ogonkov commented Dec 18, 2020

Summary

Proposed change:

Replace Travis w/ GH Actions

  • Test will run only when *.js files or package.json changed
  • codecov package replaced with official Codecov GitHub Action
  • npm modules cached

Closes #1333.

Checklist

I've completed the checklist below to ensure I didn't forget anything. This makes reviewing this PR as easy as possible for the maintainers. And it gets this change released as soon as possible.

@ogonkov ogonkov marked this pull request as draft December 18, 2020 15:34
@ogonkov ogonkov marked this pull request as ready for review December 18, 2020 15:44
@codecov-io
Copy link

codecov-io commented Dec 18, 2020

Codecov Report

Merging #1334 (cefad41) into master (aa9e5b9) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1334      +/-   ##
==========================================
- Coverage   89.69%   89.65%   -0.04%     
==========================================
  Files          22       22              
  Lines        3046     3046              
==========================================
- Hits         2732     2731       -1     
- Misses        314      315       +1     
Impacted Files Coverage Δ
nunjucks/src/node-loaders.js 64.17% <0.00%> (-1.50%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa9e5b9...cefad41. Read the comment docs.

@ogonkov
Copy link
Contributor Author

ogonkov commented Dec 23, 2020

@fdintino can you look at this changes please?

@fdintino fdintino merged commit cefad41 into mozilla:master Dec 27, 2020
@ogonkov ogonkov deleted the replace_travis branch December 28, 2020 07:05
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.

Travis CI free usage ends Dec 3; mozilla repos should switch to other CI platforms
3 participants