Merge maintenance 2.15.x following the 2.15.5 release #7662
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…values inferred (pylint-dev#7627) Add a keyword-only `compare_constants` argument to `safe_infer`.
When the dictionary has served its purpose (making plugin loading pre-and-post init a consistent behaviour), we swap it for bools indicating whether or not a module was loaded. We don't currently use the bools, but it seemed a sensible choice. The main idea is to make the dictionary fully pickle-able, so that when dill pickles the linter for multiprocessing, it doesn't crash horribly. Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com> Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
cdce8p
added
Maintenance
Release 2.15.5
Pull Request Test Coverage Report for Build 3300574766
💛 - Coveralls |
…-maintenance-2.15.5
cdce8p
force-pushed
the
merge-maintenance-2.15.5
branch
from
f6404a5
to
06ac359
Compare
Merged
Comment on lines
+264
to
+267
| "dmrtzn@gmail.com": { | ||
| "mails": ["dmrtzn@gmail.com"], | ||
| "name": "Daniel Mouritzen" | ||
| }, |
There was a problem hiding this comment.
Was it required (=crash of contributor-txt) ?
There was a problem hiding this comment.
I don't think so. I added it when I was trying to understand what contrbutor-txt was doing.
There was a problem hiding this comment.
Let's not bother removing it, this contributor might use another email later.
Pierre-Sassoulas
approved these changes
Oct 22, 2022
|
Congrats on your first release @cdce8p !! |
Thanks 😄 You all did an amazing work laying the foundation to make the process easier. The fact that it took me only a day to get up to speed is awesome. Even if we can still improve some smaller things 😉 I'll still leave the minor release to you for the time being though. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ref: #7660