Remove Travis CI and update docs for GitHub Actions #1225
Conversation
This fixes remotestorage#1222 by removing all references to Travis CI (besides in `CHANGELOG.md`) and replaces the references with information about the recently added GitHub Actions. This Pull Request is in preperation so when the project is ready to remove Travis CI it should be an easy click of `Merge Pull Request`.
|
Very nice experience so far. Travis has become so slow over the years, but Actions was pretty fast for a new PR just now! 👌 Just wondering if we can add the word "node.js" or "nodejs" before the node version in the check title somehow, so that it's clear what the numbers mean...
|
Previously, checks would display `test-and-lint/build (10)`. After this change, they should display a more user-friendly name of `test-and-build/node.js (10)`
|
Awesome, glad you're liking it! I went ahead and just made the change in this PR as a new commit. Please click
|
|
Perfect! 👍 @remotestorage/core Travis took longer than 10 minutes for a new PR yesterday. So I think we should just merge this PR and be done with Travis for now. WDYT? |
|
Nice, this looks great. And I like that you don't have to load a whole nother site in case a test fails. +1 from me. |
This fixes #1222 by removing all references to Travis CI (besides in
CHANGELOG.md) and replaces the references with information about the recently added GitHub Actions.This Pull Request is in preperation so when the project is ready to remove Travis CI it should be an easy click of
Merge Pull Request.