FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

GitHub Actions by aminya · Pull Request #684 · atom/language-javascript · GitHub

This repository was archived by the owner on Dec 15, 2022. It is now read-only.
/ language-javascript Public archive

GitHub Actions - #684

Merged
darangi merged 1 commit into
atom:masterfrom
aminya:github-actions
Dec 11, 2020
Merged

darangi merged 1 commit into
atom:masterfrom
aminya:github-actions

Conversation

aminya commented Nov 12, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description of the change

Use GitHub Actions instead of Travis and Appveyor.

Benefits

  • GitHub Actions is much faster than the other CI services and allows parallel runs
  • GitHub Actions it is much easier to configure.
  • Atom is a GitHub product, so we should use GitHub Actions.
  • Travis is making its services limited and monetary.
    https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing

Verification

The CI passes: https://github.com/aminya/language-javascript/actions/runs/359738214

Drawbacks

none

Release Notes

N/A

Copy link
Copy Markdown
Contributor

(Probably a silly question)
Is AppVeyor removed because it's redundant?

aminya commented Nov 12, 2020

Copy link
Copy Markdown
Contributor Author

Appveyor was being used for testing the code on Windows. But now GitHub Actions has Windows support, we don't need Appveyor anymore.

P.S: Because I don't have access to this organization, I cannot set the "ignore build if no appveyor.yml" option, which disables Appveyor when appveyor.yml is deleted. So I have to use a dummy appveyor.yml to achieve the same thing.

icecream17 mentioned this pull request Nov 21, 2020
darangi merged commit a5f8406 into atom:master Dec 11, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL