Skip to content

Commit

Permalink
Make tabulate a dependency for all pip install
Browse files Browse the repository at this point in the history
- But not for using the slurm2sql main program only
  • Loading branch information
rkdarst committed Aug 11, 2024
1 parent aa3c45c commit 0738579
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install pytest
python -m pip install .[cli]
python -m pip install .
- name: pytest
run: |
pytest test.py
8 changes: 4 additions & 4 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -24,16 +24,16 @@ forms than built-in Slurm commands. This is especially useful for
``sacct``. You can design your own tools like this.



Installation
------------

Normal ``pip`` installation, name ``slurm2sql`` or ``slurm2sql[cli]``
for the command line programs. This installs the library and command
line programs.
Normal ``pip`` installation, name ``slurm2sql`` for the command line
programs. This installs the library and command line programs.

::

pip install slurm2sql[cli]
pip install slurm2sql

There is only a single file with no depencecies for the core
``slurm2sql`` library (which could also be manually downloaded - HPC,
Expand Down
8 changes: 4 additions & 4 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,10 @@
#packages=setuptools.find_packages(),
py_modules=["slurm2sql"],
keywords='slurm sqlite3',
python_requires='>= 2.7, >=3.4',
extras_require = {
'cli': ["tabulate"],
},
python_requires='>= 3.6', # f-strings, if you need older then ask
install_requires = [
"tabulate",
],
entry_points={
'console_scripts': [
'slurm2sql=slurm2sql:main',
Expand Down

0 comments on commit 0738579

Please sign in to comment.