Skip to content
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: KeyError: 'home-page' #1127

Closed
wants to merge 1 commit into from
Closed

fix: KeyError: 'home-page' #1127

wants to merge 1 commit into from

Conversation

JohnVillalovos
Copy link

This is the simplest fix to move forward on resolving the issue a KeyError occurring.

Error seen is like this:
$ python -m twine check dist/*
Traceback (most recent call last):
File "/py/lib/python3.8/runpy.py", line 185, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/py/lib/python3.8/runpy.py", line 144, in _get_module_details
return _get_module_details(pkg_main_name, error)
File "/py/lib/python3.8/runpy.py", line 111, in _get_module_details
import(pkg_name)
File "/venv/lib/python3.8/site-packages/twine/init.py", line 40, in
uri = metadata["home-page"]
File "/venv/lib/python3.8/site-packages/importlib_metadata/_adapters.py", line 54, in getitem
raise KeyError(item)
KeyError: 'home-page'

Related: #977

@JohnVillalovos
Copy link
Author

Simplistic alternative to #1124 and #1126

@henryiii
Copy link
Contributor

The fix in #1126 works fine, you can see the docs job passing. There are other changes needed to get the CI passing.

This is the simplest fix to move forward on resolving the issue a
`KeyError` occurring.

Error seen is like this:
$ python -m twine check dist/*
Traceback (most recent call last):
  File "/py/lib/python3.8/runpy.py", line 185, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/py/lib/python3.8/runpy.py", line 144, in _get_module_details
    return _get_module_details(pkg_main_name, error)
  File "/py/lib/python3.8/runpy.py", line 111, in _get_module_details
    __import__(pkg_name)
  File "/venv/lib/python3.8/site-packages/twine/__init__.py", line 40, in <module>
    __uri__ = metadata["home-page"]
  File "/venv/lib/python3.8/site-packages/importlib_metadata/_adapters.py", line 54, in __getitem__
    raise KeyError(item)
KeyError: 'home-page'

Related: #977
@JohnVillalovos
Copy link
Author

Looks like #1126 is working. Closing this.

wzjin2017 added a commit to FedGraph/fedgraph that referenced this pull request Jun 26, 2024
yh-yao pushed a commit to FedGraph/fedgraph that referenced this pull request Jun 26, 2024
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.

2 participants