Skip to content

Conversation

emlowe
Copy link
Contributor

@emlowe emlowe commented Sep 30, 2025

Keeping up-to-date with the version used by dependabot.

Also tweak the license component per poetry version update (same license, just slightly different format)

@emlowe emlowe requested review from altendky and a team as code owners September 30, 2025 15:41
Copy link

socket-security bot commented Sep 30, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedpypi/​poetry@​2.1.1 ⏵ 2.2.089 -11100100100100

View full report

Copy link

coveralls-official bot commented Sep 30, 2025

Pull Request Test Coverage Report for Build 18170113404

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 15 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.006%) to 91.156%

Files with Coverage Reduction New Missed Lines %
chia/timelord/timelord.py 3 71.62%
chia/full_node/full_node.py 6 86.9%
chia/server/node_discovery.py 6 81.22%
Totals Coverage Status
Change from base Build 18135359927: 0.006%
Covered Lines: 102853
Relevant Lines: 112671

💛 - Coveralls

Copy link
Contributor

@altendky altendky left a comment

Choose a reason for hiding this comment

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

update the comment in poetry.lock as well?

Co-authored-by: Kyle Altendorf <[email protected]>
@emlowe emlowe requested a review from altendky October 1, 2025 17:19
@emlowe emlowe added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Oct 1, 2025
Copy link
Contributor

@altendky altendky left a comment

Choose a reason for hiding this comment

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

oh, i see. the lock comment was already updated

@emlowe
Copy link
Contributor Author

emlowe commented Oct 1, 2025

oh, i see. the lock comment was already updated

It was probably some Dependabot update, as DP has been on 2.2.0 for a week

@Quexington Quexington added the ready_to_merge Submitter and reviewers think this is ready label Oct 3, 2025
@pmaslana pmaslana merged commit 8e8db78 into main Oct 6, 2025
355 of 357 checks passed
@pmaslana pmaslana deleted the EL.poetry-update branch October 6, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes ready_to_merge Submitter and reviewers think this is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants