-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: Fix version, update elvis config and variable renaming #17
Conversation
I think it's failing because of https://github.com/kivra/id_token/blob/master/Makefile#L36 (head 😖); your the change is sane, though. Would it be Ok for you to use the |
Feel to free to help me out here @kivra-pauoli :) |
A proper fix would probably be to not avoid it, but 1. that'd change code, which is not our priority here, 2. that might not be easy to achieve when macros are involved
e2028f0
to
fbc814e
Compare
Instead of an actual fix we work around it by allowing it; on the other hand I'm not sure why it exists in the code but it's not our priority to check that now
@zigge-zagge, we're back in business. Feel free to merge; you have my blessing already. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All this means the ruleset
rules are applied as default. There's a new ruleset (strict
) that we can try in the future (that includes more stricter default rules).
Nice work @kivra-pauoli! |
This won't create a release but I think that's better than to let a change like this propagate :)