GeezerVison

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately we couldn't find an account corresponding to your GitHub username on bugs.python.org (b.p.o) to verify you have signed the CLA (this might be simply due to a missing "GitHub Name" entry in your b.p.o account settings). This is necessary for legal reasons before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

When your account is ready, please add a comment in this pull request
and a Python core developer will remove the CLA not signed label
to make the bot check again.

Thanks again for your contribution, we look forward to reviewing it!

I signed the CLA, Thank you for reviewing.

@eric-wieser Thanks, I fixed it now.

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@Windsooon, please resolve the merge conflict and address the code review comments. Thank you!

To make it easier, how about

"... integer or a floating point number or anything defining __abs__. If the argument is a complex number, its magnitude is returned."

Sorry for the mistake. I just fixed it.

To make it easier, how about

"... integer or a floating point number or anything defining __abs__. If the argument is a complex number, its magnitude is returned."

Yes, by using

:meth:`__abs__`

instead of a raw abs.

I'm not a native english speaker but I think the first or is now redundent, what about:

The argument may be an integer, a floating point number, or anything defining :meth:`__abs__`.

I'd maybe prefer a bit object over anything, like:

The argument may be an integer, a floating point number, or an object implementing :meth:`__abs__`.

@JulienPalard Thank you. I updated the PR based on your suggestion.

@Windsooon could you resolve the conflict?
I would personally do it using git fetch upstream; git rebase upstream/master (my python/cpython remote being named upstream), as your PR is based on a commit from 2018.

rebase commonly messes up branches and PRs with the cpython workflow. Did you do git merge in master before updating the branch? git merge upstream/master usually works. There might be a merge conflict, but not likely with a narrow PR like this one. You could still try merge on your local branch if not deleted yet.

@terryjreedy Thank you. I just delete the old branch. However, I don't know why my PR will conflict with the current master branch. I will check it out later.