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

Fix braces #20

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@ambs

ambs commented Aug 26, 2015

Hi,

Perl 5.22 complains on braces unescaped on substitutions.
This fixes that.

Simple but useful fix, I hope.

Best
ambs

@mat813

This comment has been minimized.

Show comment
Hide comment
@mat813

mat813 May 19, 2016

And 5.25.1 makes it fatal. 👍

mat813 commented May 19, 2016

And 5.25.1 makes it fatal. 👍

@chrisg86

This comment has been minimized.

Show comment
Hide comment
@chrisg86

chrisg86 Oct 4, 2016

Are there any plans on merging this pr? It's been over a year by now.

chrisg86 commented Oct 4, 2016

Are there any plans on merging this pr? It's been over a year by now.

@ambs

This comment has been minimized.

Show comment
Hide comment
@ambs

ambs Oct 4, 2016

Probably this is enough time and we should just ask modules@perl.org to overtake this module and fix this issue... @mirod?

ambs commented Oct 4, 2016

Probably this is enough time and we should just ask modules@perl.org to overtake this module and fix this issue... @mirod?

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