Skip to content

Re-enable CI#171

Merged
zerowidth merged 3 commits into
github:mainfrom
brasic:main
Jan 3, 2022
Merged

Re-enable CI#171
zerowidth merged 3 commits into
github:mainfrom
brasic:main

Conversation

@brasic
Copy link
Copy Markdown
Contributor

@brasic brasic commented Dec 23, 2021

CI doesn't appear to be configured on this repo anymore, which makes it difficult to accept pull requests. This PR:

  • Adds rake as a test runner with test as the default task.
  • Removes the coveralls gem since its configuration was lost.
  • Sets up Actions to run CI. I've added a matrix that includes all supported MRI versions.

(see an example PR with this config here)

Carl Brasic added 3 commits December 23, 2021 14:10
The configuration to publish code coverage metrics to coveralls.io is no
longer present anywhere, so we may as well remove this dep.
@zerowidth zerowidth merged commit c7c5e69 into github:main Jan 3, 2022
@zerowidth
Copy link
Copy Markdown
Contributor

Rad, thanks for setting this up.

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.

2 participants