Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

GitHub actions #899

Merged
merged 6 commits into from
Mar 4, 2021
Merged

Conversation

aminya
Copy link
Contributor

@aminya aminya commented Aug 23, 2020

Description of the change

Use GithubActions instead of Travis and Appveyor. GitHubActions is much faster than the other CI services, and it is much easier to configure.

Verification

The CI passes: https://github.com/atom-ide-community/apm/actions/runs/413583758

Drawbacks

none

Release Notes

N/A

@aminya
Copy link
Contributor Author

aminya commented Sep 3, 2020

I added x86 support to GitHub's setup-node. actions/setup-node#194

If they merge that PR sooner than you merge this PR, I will update this branch. Otherwise, it will be a follow-up.

@aminya
Copy link
Contributor Author

aminya commented Dec 10, 2020

My PR is finally merged. Now, we can use the official setup node with x86 support inside GitHub Actions.
actions/setup-node#194

I have updated this PR accordingly.

Copy link
Contributor

@DeeDeeG DeeDeeG left a comment

Choose a reason for hiding this comment

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

👍

Note to maintainers: Please consider disabling AppVeyor and Travis for this repository and activating GitHub Actions when merging this.

I like using GitHub Actions for this repo (and this is what we did at the community fork) because it's much easier for users to get GitHub Actions running than to get Travis and AppVeyor both running.

.github/workflows/CI.yml Outdated Show resolved Hide resolved
@darangi
Copy link
Contributor

darangi commented Mar 4, 2021

Thanks @aminya

@darangi darangi merged commit a6f65c4 into atom:master Mar 4, 2021
@aminya aminya deleted the github-actions-4upstream branch March 6, 2021 07:12
@aminya aminya restored the github-actions-4upstream branch March 6, 2021 07:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants