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

OSS Requrements, CI configuration #6

Merged
merged 7 commits into from
Jun 1, 2019
Merged

OSS Requrements, CI configuration #6

merged 7 commits into from
Jun 1, 2019

Conversation

0exp
Copy link
Member

@0exp 0exp commented May 25, 2019

  • actualized license years
  • OSS mail contact
  • travis recofnigration
  • rubocop development dependency
  • bin/console utility

@0exp 0exp requested a review from tycooon May 25, 2019 15:51
@0exp 0exp self-assigned this May 25, 2019
@0exp 0exp changed the title OSS Requrements OSS Requrements, CI configuration May 25, 2019
@0exp 0exp requested review from akxcv and nesaulov May 25, 2019 16:51
Copy link

@akxcv akxcv left a comment

Choose a reason for hiding this comment

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

lgtm

.rubocop.yml Outdated
@@ -3,7 +3,7 @@ inherit_gem:

AllCops:
DisplayCopNames: true
TargetRubyVersion: 2.5
TargetRubyVersion: 2.6
Copy link
Member

Choose a reason for hiding this comment

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

I think that target rubocop version should be the minimum supported version of MRI

Copy link
Member

Choose a reason for hiding this comment

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

+1

Copy link
Member

@tycooon tycooon left a comment

Choose a reason for hiding this comment

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

Please fix the travis build.

polist.gemspec Show resolved Hide resolved
.rubocop.yml Outdated
@@ -3,7 +3,7 @@ inherit_gem:

AllCops:
DisplayCopNames: true
TargetRubyVersion: 2.5
TargetRubyVersion: 2.6
Copy link
Member

Choose a reason for hiding this comment

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

+1

@0exp 0exp requested review from nesaulov and tycooon and removed request for nesaulov May 27, 2019 20:10
@tycooon tycooon merged commit 69702b7 into master Jun 1, 2019
@tycooon tycooon deleted the oss-requirements branch June 1, 2019 13:00
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.

None yet

4 participants