Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dni
Copy link
Member

@dni dni commented Jun 26, 2025

some packages are not updated.

  • sqlalchemy
  • pydantic
  • lnurl
  • httpx required by lnurl

interestingly updating the linters now required formatting.

removed lnbits/requestvars.py it also had a new weird linting error and by checking it out, it seemed to be used in only 1 place, i think it was some legacy code.

had to change import from websockets library, to add a legacy namespace

@codecov
Copy link

codecov bot commented Jun 26, 2025

Codecov Report

Attention: Patch coverage is 86.45833% with 13 lines in your changes missing coverage. Please review.

Project coverage is 61.03%. Comparing base (974ee26) to head (f9bda24).
Report is 2 commits behind head on dev.

Files with missing lines Patch % Lines
lnbits/wallets/strike.py 22.22% 7 Missing ⚠️
lnbits/wallets/breez.py 0.00% 2 Missing ⚠️
lnbits/helpers.py 66.66% 1 Missing ⚠️
lnbits/wallets/lnpay.py 66.66% 1 Missing ⚠️
lnbits/wallets/lntips.py 66.66% 1 Missing ⚠️
lnbits/wallets/zbd.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #3221      +/-   ##
==========================================
- Coverage   61.03%   61.03%   -0.01%     
==========================================
  Files         111      110       -1     
  Lines       14308    14320      +12     
==========================================
+ Hits         8733     8740       +7     
- Misses       5575     5580       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dni dni changed the title chore: update python packages chore: update python packages + formatting + cleanup Jun 26, 2025
@dni dni requested a review from motorina0 June 26, 2025 13:31
@dni dni force-pushed the update-packages branch 2 times, most recently from 871dbf9 to 2966cfa Compare June 30, 2025 12:01
@motorina0 motorina0 merged commit 8337351 into dev Jun 30, 2025
44 checks passed
@motorina0 motorina0 deleted the update-packages branch June 30, 2025 12:53
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