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

updated changelog from 1.57 rt 105983 #27

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@brainbuz
Contributor

brainbuz commented Jul 25, 2015

This is the most trivial bug I could find to fix, it is just an update to the changelog entry for 1.57 to indicate that perl 5.6 is no longer supported. (I don't think this warrants a further revision to the changelog to indicate that a prior entry was corrected)

The policy as I understand it is for upstream modules such as this to support at least Perl 5.8. Last month I helped David Golden maintain 5.8 support in Path::Tiny for my pull request challenge contribution.

I was looking at uri for the pull request challenge, there are 45 rtcpan bugs in either new or open plus a few more reported on github. In reviewing them I believe that many can be either marked closed or rejected, which makes it more confusing as someone just volunteering a few hours to help out to pick a task to work on.

@karenetheridge

This comment has been minimized.

Show comment
Hide comment
@karenetheridge

karenetheridge Jul 29, 2015

Member

Thanks, I will merge this soon!

I was looking at uri for the pull request challenge, there are 45 rtcpan bugs in either new or open plus a few more reported on github. In reviewing them I believe that many can be either marked closed or rejected, which makes it more confusing as someone just volunteering a few hours to help out to pick a task to work on.

Please feel free to comment on each ticket that you look at if you discover something (e.g. a fix in a previous release), as this will save time for the next person to look at the ticket, and enable it to be closed more quickly.

Member

karenetheridge commented Jul 29, 2015

Thanks, I will merge this soon!

I was looking at uri for the pull request challenge, there are 45 rtcpan bugs in either new or open plus a few more reported on github. In reviewing them I believe that many can be either marked closed or rejected, which makes it more confusing as someone just volunteering a few hours to help out to pick a task to work on.

Please feel free to comment on each ticket that you look at if you discover something (e.g. a fix in a previous release), as this will save time for the next person to look at the ticket, and enable it to be closed more quickly.

@karenetheridge karenetheridge self-assigned this Jul 29, 2015

@karenetheridge

This comment has been minimized.

Show comment
Hide comment
@karenetheridge

karenetheridge Feb 7, 2016

Member

Thanks, merged (with small edits)!

Member

karenetheridge commented Feb 7, 2016

Thanks, merged (with small edits)!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment