Skip to content

Conversation

@jylamont
Copy link
Contributor

@jylamont jylamont commented Apr 17, 2024

We previously used TravisCI to run specs. This PR replaces this with a Github Action. It also removes the version locks on development dependencies as they are very old.

@jylamont jylamont force-pushed the 04-17-improve_added_github_action_ci branch from 939b9aa to ea3dfee Compare April 17, 2024 22:45
@jylamont jylamont requested a review from edubkendo April 17, 2024 22:47
@jylamont jylamont marked this pull request as ready for review April 17, 2024 22:47
Copy link

@edubkendo edubkendo left a comment

Choose a reason for hiding this comment

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

Looks good! It's a shame there's no way to just hand circleCI a list of ruby versions and then write the boilerplate for running specs once.

Copy link
Contributor Author

Good point @edubkendo, there very likely is. I'll take a look before merging.

@jylamont jylamont force-pushed the 04-17-improve_added_github_action_ci branch from ea3dfee to e4579a9 Compare April 18, 2024 00:02
Copy link
Contributor Author

jylamont commented Apr 18, 2024

Merge activity

  • Apr 17, 8:03 PM EDT: @jylamont started a stack merge that includes this pull request via Graphite.
  • Apr 17, 8:10 PM EDT: Graphite rebased this pull request as part of a merge.
  • Apr 17, 8:11 PM EDT: @jylamont merged this pull request with Graphite.

@jylamont jylamont force-pushed the 04-17-chore_run_rubocop_over_existing_code branch from dc29935 to a35b33a Compare April 18, 2024 00:07
Base automatically changed from 04-17-chore_run_rubocop_over_existing_code to master April 18, 2024 00:09
@jylamont jylamont force-pushed the 04-17-improve_added_github_action_ci branch from e4579a9 to a3c498e Compare April 18, 2024 00:10
@jylamont jylamont merged commit 1f26a6e into master Apr 18, 2024
@jylamont jylamont deleted the 04-17-improve_added_github_action_ci branch April 18, 2024 00:11
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