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

Use GitHub Actions for CI and Prepare to use GitHub Package Registry #4600

Merged
merged 13 commits into from
Aug 12, 2019

Conversation

lildude
Copy link
Member

@lildude lildude commented Aug 11, 2019

Description

It's been an exciting few months for GitHub, especially last week with the introduction of GitHub Actions adding support for CI/CD.

This PR switches Linguist from using Travis for CI to GitHub Actions 🎉 . Wanna see it in action? https://github.com/github/linguist/runs/190621762

At the same time, I'm adding support for publishing packages to the GitHub Package Registry so we can use it from the next release onwards. One day we'll also be able to take advantage of GitHub Actions to create and publish the gem automatically too, but can't right now due to reasons I can't go into.

Checklist removed as it doesn't apply.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
Copy link
Collaborator

@Alhadis Alhadis left a comment

Choose a reason for hiding this comment

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

First half of this PR reads like an advert, but otherwise LGTM. 👍

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@lildude
Copy link
Member Author

lildude commented Aug 12, 2019

All feedback implemented @Alhadis. Thanks 🙇. Wanna take another peek? And feed free to suggest any other changes that have been on your mind; we've not touch the CI side of things for many many years.

CONTRIBUTING.md Outdated Show resolved Hide resolved
@Alhadis
Copy link
Collaborator

Alhadis commented Aug 12, 2019

Only one other thing I've picked up on, but other than that, 👍 from me!

@lildude
Copy link
Member Author

lildude commented Aug 12, 2019

I should also give a nod to the AWS Lambda that powers the badge - https://github.com/CultureHQ/github-actions-badge/blob/master/README.md

@lildude lildude merged commit b475846 into master Aug 12, 2019
@lildude lildude deleted the lildude/use-actions-for-ci branch August 12, 2019 13:04
@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants