-
-
Notifications
You must be signed in to change notification settings - Fork 111
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: Ensure tab characters in policy files are parsed as white-space #324
Conversation
@sagilio please review |
@janpieterz plz add the unit test, thx! |
@hsluoyz sorted! |
@janpieterz fix: |
@hsluoyz should be fixed now |
@janpieterz the benchmark CI is too slow. Can you remove it? |
@hsluoyz you mean remove it from running as a pull request check? |
@janpieterz remove |
Sure, done here |
🎉 This PR is included in version 2.0.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
If the policy file has tabs in it, the change to use the CSV parser meant that the policies were parsed incorrect.
I have a unit test that proves this that I can add if wanted (seems very low level but might be good to ensure regressions).