-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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(metricpass): Remove python logic compatibility #13791
Conversation
this is the old affected version.
this is the fix-artefact:
this is the old affected version.
this is the fix-artefact:
Looks good to me |
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.
This is a breaking change and should be declared as such in the Changelog! Please also note that this was an explicit request by Dave back when I added this.
This being said, I'm fine with removing this as we can get rid of another regex pitfall and will approve when the breaking change is documented in the Changelog.
PR updated with change log entry!
Ah ok thanks for the history |
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.
Thanks for taking care of this @powersj!
Download PR build artifacts for linux_amd64.tar.gz, darwin_amd64.tar.gz, and windows_amd64.zip. 📦 Click here to get additional PR build artifactsArtifact URLs |
fixes: #13790