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

cmd: import hostlist module, bump version #37

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cmoussa1
Copy link
Member

@cmoussa1 cmoussa1 commented Sep 4, 2024

Problem

Building the hostlist package with the commands listed in the start page does not allow for actual use of any of the hostlist commands and instead raises an AttributeError complaining that it can't find any of the functions.


This PR adds the import of the hostlist package to __init__.py so that the command-line interface can actually utilize the functions defined in hostlist.py and a user can run things like hostlist -e node[1-4] instead of python cla_hostlist.py -e node[1-4].

It also bumps the version of the py-hostlist package to 0.1.0.

Problem: Building the hostlist package with the commands listed in the
start page does not allow for actual use of any of the 'hostlist *'
commands and instead raises an AttributeError complaining that it can't
find any of the functions.

Add the import of the hostlist package to __init__.py so that the
command-line interface can actually utilize the functions defined in
hostlist.py and a user can run things like 'hostlist -e node[1-4]'
instead of 'python cla_hostlist.py -e node[1-4]'.

Bump the version of py-hostlist to 0.1.0.
@cmoussa1 cmoussa1 added the bug-fix fixes something that isn't working label Sep 4, 2024
Problem: The test installation of the hostlist package explicitly
specifies the version number of the hostlist package, but this will no
longer apply when the version number is bumped up.

Change the "pip install" command to instead just use a wildcard so that
we don't have to change the name of the package every time the version
number is bumped.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix fixes something that isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant