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

Remove TODO to see where it's an issue #29

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

atoomic
Copy link
Collaborator

@atoomic atoomic commented Apr 20, 2020

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling f6f7c0a on atoomic:todo into 801b565 on garu:master.

@garu
Copy link
Owner

garu commented Apr 21, 2020

Hey @atoomic! Shouldn't we check first on cpantesters if any reports have failed the TODO block? Otherwise, if there is an issue, people won't be able to install Clone without forcing.

@atoomic
Copy link
Collaborator Author

atoomic commented Apr 21, 2020

yeap this one is not merged, we clearly need to check cpan tester with the release from today with the support for cow being disabled

@garu
Copy link
Owner

garu commented Oct 18, 2022

@atoomic ping! As @haarg mentioned in #39, we need to first ship t/cow.t and only then inspect cpantesters to see where it passes or fails. I'm just waiting on you guys reviewing #40 so I can merge and push a new version to cpan (with the updated MANIFEST so it includes t/cow.t) <3

@atoomic atoomic marked this pull request as draft October 18, 2022 19:30
@garu
Copy link
Owner

garu commented Oct 20, 2022

Clone 0.46 is pushed!

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.

3 participants