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
Setup circleci #128
Setup circleci #128
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with error in Travis ignored
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed for Data Exfiltrations
How about removing Travis and AppVeyor configs? Both seem not working anyway, and you could test Windows in CircleCI as needed too. |
td-client.gemspec
Outdated
@@ -26,6 +26,9 @@ Gem::Specification.new do |gem| | |||
gem.add_development_dependency 'simplecov', '>= 0.5.4' | |||
gem.add_development_dependency 'rake' | |||
gem.add_development_dependency 'yard' | |||
if RUBY_VERSION >= '3.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is interpreted by a particular Ruby version on gem push
, which means the dependency definition will be changed what Ruby version you'll use for gem push
.
You should simply declare the dependency regardless of the Ruby version if you really want to make sure it's specified as a development dependency when you use the gem from Ruby 3.0. Development dependencies of released packages probably don't really matter though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah I see. I have wrong understanding of RUBY_VERSION in gemspec. Thanks for catching it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think win_ruby
could be a separate PR and it'd be nice to make master CI green first. But either way this is a great job.
Travis CI and AppVeyor were removed with PR #128. Note that the current CI (Circle CI) page is not public, so we cannot add its badge.
Travis CI and AppVeyor were removed with PR #128. Note that the current CI (Circle CI) page is not public, so we cannot add its badge.
Travis CI and AppVeyor were removed with PR #128. Note that the current CI (Circle CI) page is not public, so we cannot add its badge.
PR to setup Circle CI: