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 #442

Merged
merged 4 commits into from
May 26, 2023
Merged

Conversation

mlarraz
Copy link
Contributor

@mlarraz mlarraz commented Apr 25, 2023

Link to build on my fork: https://github.com/mlarraz/attr_encrypted/actions/runs/4798509264/

Not sure what the current story is with Travis, but Github Actions is free and starts quickly.

@joshbranham
Copy link
Member

Link to build on my fork: https://github.com/mlarraz/attr_encrypted/actions/runs/4798509264/

Not sure what the current story is with Travis, but Github Actions is free and starts quickly.

I was open to switching to Github Actions as well, but just haven't prioritized it. When we started contributing, @mvastola reached out to Travis to get credits added. Given that process is manual and time delayed, I would vote to switch to GitHub Actions.

Note: We should probably keep the ActiveRecord 5.x testing if it's not too complicated, given folks out there likely still running that. Happy to discuss removing it in another PR down the line.

@mlarraz
Copy link
Contributor Author

mlarraz commented Apr 25, 2023

Rails 5.1 and 5.2 are not compatible with Ruby 3, so I will have to exclude those combinations from the build matrix if you plan to keep support

@joshbranham
Copy link
Member

joshbranham commented Apr 25, 2023

Yup I did that as well for Travis here #440

@joshbranham
Copy link
Member

@mvastola I am in favor of dropping Travis for GitHub Actions as well. Let me know your thoughts, if so we can merge this and close #440

Copy link
Member

@mvastola mvastola left a comment

Choose a reason for hiding this comment

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

@joshbranham I don't have much experience with GitHub actions. I wouldn't mind staying on Travis, but if you think it'd be better to switch, you have my sign-off.

@mvastola
Copy link
Member

@joshbranham wait, is there an issue with our credits? It looks like this ran on Travis 3 days ago. FWIW, they said something about automatic credits (I'd have to dig up the email). This project had just lapsed for so long they were hesitant to give us any. The way we left off I'm pretty sure they suggested automatic credits were likely if we stay active.

Anyway, I'm deferring to you. Just approved this so you're able to merge should you deem it prudent.

@joshbranham
Copy link
Member

@joshbranham wait, is there an issue with our credits? It looks like this ran on Travis 3 days ago. FWIW, they said something about automatic credits (I'd have to dig up the email). This project had just lapsed for so long they were hesitant to give us any. The way we left off I'm pretty sure they suggested automatic credits were likely if we stay active.

Anyway, I'm deferring to you. Just approved this so you're able to merge should you deem it prudent.

There isn't a current issue with credits, I'm just saying GitHub actions is free for open source and pretty defacto these days.

@joshbranham joshbranham merged commit 6bab9de into attr-encrypted:master May 26, 2023
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.

3 participants