Skip to content

Conversation

@knaaptime
Copy link
Member

No description provided.

python -m build
twine check --strict dist/*
pip install setuptools wheel twine jupyter urllib3 pandas pyyaml
python setup.py sdist bdist_wheel
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jGaboardi does this line with setup.py still work with pyproject.toml? I thought you had to use pip install . ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See here in spaghetti's release workflow.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah ok thanks. I snagged the last one from spopt

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have not had enough time recently to sync everything 😰

@codecov
Copy link

codecov bot commented Aug 23, 2023

Codecov Report

Merging #220 (4b17523) into main (57acfb8) will increase coverage by 10.50%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #220       +/-   ##
===========================================
+ Coverage   71.38%   81.88%   +10.50%     
===========================================
  Files          67       66        -1     
  Lines        2628     2291      -337     
===========================================
  Hits         1876     1876               
+ Misses        752      415      -337     

@knaaptime knaaptime merged commit f431b1d into pysal:main Aug 23, 2023
@knaaptime knaaptime deleted the release_action branch August 23, 2023 18:44
@knaaptime knaaptime mentioned this pull request Aug 23, 2023
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