FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Bad params object[] precedence by lostmsu · Pull Request #1213 · pythonnet/pythonnet · GitHub

Bad params object[] precedence - #1213

Merged
filmor merged 1 commit into
pythonnet:backports-2.5from
losttech:PR/BadParamsObjectPrecedence
Sep 14, 2020
Merged

Bad params object[] precedence#1213
filmor merged 1 commit into
pythonnet:backports-2.5from
losttech:PR/BadParamsObjectPrecedence

Conversation

lostmsu commented Aug 29, 2020

Copy link
Copy Markdown
Member

What does this implement/fix? Explain your changes.

A bug in ArgsPrecedence caused parameters of type object[] to have the highest precedence, which affected params object[] handling, and also preferred params object[] to params string[] when a string argument was given.

Does this close any currently open issues?

#1211

Any other comments?

...

Checklist

Check all those that are applicable and complete.

  • Make sure to include one or more tests for your change
  • If an enhancement PR, please create docs and at best an example
  • Add yourself to AUTHORS
  • Updated the CHANGELOG

codecov-commenter commented Aug 29, 2020
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #1213 into backports-2.5 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           backports-2.5    #1213   +/-   ##
==============================================
  Coverage          86.53%   86.53%           
==============================================
  Files                  1        1           
  Lines                297      297           
==============================================
  Hits                 257      257           
  Misses                40       40           
Flag Coverage Δ
#setup_linux 64.98% <ø> (ø)
#setup_windows 72.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96adaaa...6df753f. Read the comment docs.

lostmsu commented Sep 13, 2020

Copy link
Copy Markdown
Member Author

@filmor can you review this?

This file contains hidden or 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL