Skip to content

Conversation

@bluemellophone
Copy link
Collaborator

@bluemellophone bluemellophone commented Feb 5, 2026

Now that the latest version of pyastar2d is working on PyPI, we can remove the batbot-pyastar2d fork and upload our package to PyPI.

This PR also fixes the documentation building:

Screenshot 2026-02-05 at 14 14 25 Screenshot 2026-02-05 at 14 08 39

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.4%. Comparing base (669f21c) to head (31c9aca).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main     #36   +/-   ##
=====================================
  Coverage   57.4%   57.4%           
=====================================
  Files          7       7           
  Lines       1317    1317           
=====================================
  Hits         756     756           
  Misses       561     561           
Files with missing lines Coverage Δ
batbot/batbot_cli.py 0.0% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 669f21c...31c9aca. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stoutta6
Copy link
Collaborator

stoutta6 commented Feb 5, 2026

@bluemellophone I had to modify cli.rst line 11 to ".. click:: batbot.batbot_cli:cli" due to the filename change for the CLI page to build properly. It built but with a few warnings. Did you modify this file as well?

@bluemellophone
Copy link
Collaborator Author

@bluemellophone I had to modify cli.rst line 11 to ".. click:: batbot.batbot_cli:cli" due to the filename change for the CLI page to build properly. It built but with a few warnings. Did you modify this file as well?

Good catch, can you see if you can fix the build warnings? I don't remember modifying cli.rst

Copy link
Collaborator

@stoutta6 stoutta6 left a comment

Choose a reason for hiding this comment

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

LGTM, fixed a couple warnings/errors. There are still a couple warnings that show up (related to having multiple includes of the same docs page or having multiple functions with the same name) but they don't seem to affect anything.

@stoutta6 stoutta6 merged commit 285a833 into main Feb 10, 2026
8 checks passed
@bluemellophone bluemellophone deleted the jrp/pypi-deployment branch February 10, 2026 19:42
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.

3 participants