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

Possible bug in algorithm #1

Closed
jonasbn opened this issue Aug 25, 2014 · 1 comment
Closed

Possible bug in algorithm #1

jonasbn opened this issue Aug 25, 2014 · 1 comment
Assignees
@jonasbn
Labels
bug
Milestone
Release 0.02

Comments

@jonasbn
Copy link
Owner

jonasbn commented Aug 25, 2014

I got a error report. This is the essentials from the report:

  •      Include PPI statement can be not only „use“, but also „no“, e.g. for no warnings pragma.

  •      You are carping Unable to locate package keyword if we use no instead of use. You probably want to test content use in 1st children element and existence of content in 3rd children element

Investigating...
@jonasbn jonasbn self-assigned this Aug 25, 2014
@jonasbn jonasbn added the bug label Aug 25, 2014
jonasbn added a commit that referenced this issue Aug 25, 2014
@jonasbn
Addressed issue #1. The policy did not ignore 'no' use statements,
9ae395d 
which are used for pragma, instead it died
@jonasbn jonasbn added this to the Release 0.02 milestone Aug 25, 2014
@jonasbn
Copy link
Owner Author

jonasbn commented Aug 25, 2014

Release 0.02 has just been uploaded to CPAN. Closing this issue. Thanks to Milan for reporting.

@jonasbn jonasbn closed this as completed Aug 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jonasbn jonasbn

Labels
bug
Projects
None yet
Milestone
Release 0.02
Development

No branches or pull requests
1 participant
@jonasbn