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

Avoid internal error for typoed namespace #855

Merged
merged 1 commit into from
Sep 23, 2019
Merged

Avoid internal error for typoed namespace #855

merged 1 commit into from
Sep 23, 2019

Conversation

usev6
Copy link
Contributor

@usev6 usev6 commented Feb 23, 2019

Fixes #853

@usev6
Copy link
Contributor Author

usev6 commented Feb 25, 2019

I've found #586 and I wonder whether it means that patches like this PR are not the way to go. At a first glance it looks like a couple of the open issues are about internal errors caused by syntax errors. At least some of those might be low hanging fruits and I thought about trying my luck with them. But maybe you'd rather not have additional code that checks for syntax errors?

@rodjek rodjek merged commit e9f1f28 into rodjek:master Sep 23, 2019
@usev6 usev6 deleted the issue853 branch October 1, 2019 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

puppet-lint crashes with mispelled namespace seperators
2 participants
@usev6 @rodjek