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.

Sign up for GitHub

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

Jump to bottom

Generate 1.8 warning/error #40

Open
pjotrp opened this issue Sep 25, 2012 · 5 comments
Open

Generate 1.8 warning/error #40

pjotrp opened this issue Sep 25, 2012 · 5 comments
Labels
Enhancement GCI2013 Newbie

Comments

@pjotrp
Copy link
Collaborator

pjotrp commented Sep 25, 2012

Most people code for 1.9, but users still try 1.8. We should generate a warning/error by default.
@rjpbonnal
Copy link
Owner

agree.

@homonecloco
Copy link

A bit related to this, I noticed rcov is used on several projects, but that is not supported by 1.9, there is simplecov which is supposed to be the new version.

@pjotrp
Copy link
Collaborator Author

pjotrp commented Jan 29, 2013

I think we disabled rcov before. Few people need it by default.

@homonecloco
Copy link

My bad then, I spotted it while trying to make the samtools wrapper to pass the unit tests.

@pjotrp
Copy link
Collaborator Author

pjotrp commented Jan 29, 2013

No worries. Jeweler may have stuck it back in. I made a monkey patch at the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement GCI2013 Newbie
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pjotrp @rjpbonnal @homonecloco