fix: use correct location for URL

[henryiii]

Alternative to #1124, fixing the issue rather than hiding the error. Requires #1123.

[jaraco]

I'm pretty sure the assignment of __author__ is also broken, based on the reported DeprecationWarnings.

[henryiii]

Yes, I think you are right. These dunder attributes aren't a standard anyway (a failed standard, technically), I wonder if it would be fine to just remove them?

[henryiii]

"wheel" should also be removed from pyproject.toml.

[henryiii]

Setting the metaadata version too high is now a warning instead of an error; how should that be handled? Covert to error, or just test for the warning?

[henryiii]

Ahh, needs #1123. I can rebase on that.

[henryiii]

You won't actually be able to release without yanking 5.1.0, though, since

twine/.github/workflows/release.yml

Line 79 in 6fbf880

uses: "pypa/gh-action-pypi-publish@81e9d935c883d0b210363ab89cf05f3894778450"

is broken due to twine being broken! :)

[henryiii]

Hmm, looks like it's pinned, not sure why pypa/gh-action-pypi-publish#246 exists then.

[jaraco]

I've changed the base of the PR to target the branch for #1123 so that the diff is apparent. I expect GitHub to re-target main after #1123 is merged.

[jaraco]

A very similar patch was proposed in #1115. Perhaps we should accept that instead, since it was proposed first. Also, it re-uses the email module to parse the email field, which seems preferable.

[JohnVillalovos]

A very similar patch was proposed in #1115. Perhaps we should accept that instead, since it was proposed first. Also, it re-uses the email module to parse the email field, which seems preferable.

Makes sense. They went through all the effort 28 days ago to fix it...

[henryiii]

Okay, I'm going to close this to make sure that one is the one that goes in. I like the usage of the email module better too.