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

Add Github Actions CI #19

Merged
merged 4 commits into from
Oct 31, 2023
Merged

Add Github Actions CI #19

merged 4 commits into from
Oct 31, 2023

Conversation

blowmage
Copy link
Collaborator

No description provided.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
Gemfile Outdated Show resolved Hide resolved

self.readme_file = "README.rdoc"
self.history_file = "CHANGELOG.rdoc"

dependency "minitest", "~> 5.0"
dependency "rubocop", "~> 0.37", :dev
dependency "rubocop", "~> 1.57.0", :dev
Copy link

Choose a reason for hiding this comment

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

just remove 🤷

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

why remove?

Copy link

Choose a reason for hiding this comment

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

previously that was in gemfile too
... either or is fine

FYI would recommend getting rid of hoe, just adds unnecessary indirection

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

previously that was in gemfile too

ah, i understand now. the gemspec file is generated by hoe-gemspec, using the definition in Rakefile. the gemspec file is only here for the convenience of those using bundler to require the library via git, it is not used in packaging or deploying the gem

would recommend getting rid of hoe

ryan still prefers hoe, so i would expect it will stay for now

lib/minitest/rg_plugin.rb Outdated Show resolved Hide resolved
@blowmage blowmage force-pushed the add-github-actions-ci branch 2 times, most recently from 01248f2 to d307516 Compare October 31, 2023 05:40
@blowmage blowmage force-pushed the add-github-actions-ci branch from 0bfbf54 to 802d156 Compare October 31, 2023 06:02
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@blowmage blowmage force-pushed the add-github-actions-ci branch from 802d156 to f3df209 Compare October 31, 2023 06:13
Copy link

@grosser grosser left a comment

Choose a reason for hiding this comment

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

@blowmage blowmage force-pushed the add-github-actions-ci branch 2 times, most recently from 308aa13 to 8494bd3 Compare October 31, 2023 06:29
@blowmage blowmage force-pushed the add-github-actions-ci branch from 8494bd3 to 7b76b9c Compare October 31, 2023 06:32
@blowmage blowmage marked this pull request as ready for review October 31, 2023 06:38
@blowmage blowmage merged commit 332f668 into master Oct 31, 2023
7 checks passed
@blowmage blowmage deleted the add-github-actions-ci branch October 31, 2023 06:39
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