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

Require version on new command (bugfix for #35) #36

Merged
merged 1 commit into from Oct 13, 2017

Conversation

hasanen
Copy link
Contributor

@hasanen hasanen commented Oct 13, 2017

Fixes issue #35. I feel that requiring version in commands/new.rb is better than altering that plugins.rb that you suggested (at least it was easier! :D).

@@ -10,6 +10,8 @@
require_relative '../licenses'
require_relative '../plugins'
require_relative '../templater'
require_relative '../version'

Choose a reason for hiding this comment

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

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

@hasanen hasanen changed the title Require version on new command Require version on new command (bugfix for #35) Oct 13, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+2.3%) to 77.519% when pulling c5712a9 on hasanen:bugfix/35-require-version into 898c3c7 on piotrmurach:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+2.3%) to 77.519% when pulling c5712a9 on hasanen:bugfix/35-require-version into 898c3c7 on piotrmurach:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.3%) to 77.519% when pulling c5712a9 on hasanen:bugfix/35-require-version into 898c3c7 on piotrmurach:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.3%) to 77.519% when pulling c5712a9 on hasanen:bugfix/35-require-version into 898c3c7 on piotrmurach:master.

@coveralls
Copy link

coveralls commented Oct 13, 2017

Coverage Status

Coverage remained the same at 75.188% when pulling c5712a9 on hasanen:bugfix/35-require-version into 898c3c7 on piotrmurach:master.

@piotrmurach piotrmurach merged commit a5e5150 into piotrmurach:master Oct 13, 2017
@piotrmurach
Copy link
Owner

Thanks! ❤️

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