Skip to content

Expand metadata set in setup.py #19

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

Merged
merged 4 commits into from
Jun 13, 2022
Merged

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jun 12, 2022

No description provided.

ekohl added 3 commits June 12, 2022 16:09
It's already set in the long metadata, but this allows more tools to
pick it up.
This allows RPM packaging to run the tests too.
@ekohl ekohl force-pushed the update-metadata branch from 153b165 to 97d92c2 Compare June 12, 2022 14:21
@ekohl
Copy link
Member Author

ekohl commented Jun 12, 2022

Looks like 3.10 changed the help text:

       def test_obsah_argument_parser_help(fixture_dir, parser):
          path = fixture_dir / 'help.txt'
  >       assert path.read() == parser.format_help()
  E       AssertionError: assert 'usage: obsah...ge and exit\n' == 'usage: obsah...ge and exit\n'
  E         Skipping 215 identical leading characters in diff, use -v to show
  E           up
  E           
  E         - options:
  E         + optional arguments:
  E             -h, --help      show this help message and exit

Also makes the classifiers consistent with what we test in reality.
@ekohl ekohl force-pushed the update-metadata branch from 97d92c2 to 58ab490 Compare June 12, 2022 14:30
@ekohl ekohl changed the title Set license in setup.py, include tests & test on Py 3.1[01] Expand metadata set in setup.py Jun 12, 2022
@ekohl ekohl merged commit 58ab490 into theforeman:master Jun 13, 2022
@ekohl ekohl deleted the update-metadata branch June 13, 2022 10:26
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