Skip to content

fix: deduplicate dependencies#3008

Merged
ReimarBauer merged 2 commits intoOpen-MSS:developfrom
matrss:deduplicate-dependencies
Feb 27, 2026
Merged

fix: deduplicate dependencies#3008
ReimarBauer merged 2 commits intoOpen-MSS:developfrom
matrss:deduplicate-dependencies

Conversation

@matrss
Copy link
Collaborator

@matrss matrss commented Feb 24, 2026

Purpose of PR?:

Fixes #3007.

Does this PR introduce a breaking change?

If the changes in this PR are manually verified, list down the scenarios covered::

Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Does this PR results in some Documentation changes?
If yes, include the list of Documentation changes

Checklist:

  • Bug fix. Fixes #
  • New feature (Non-API breaking changes that adds functionality)
  • PR Title follows the convention of <type>: <subject>
  • Commit has unit tests

@matrss matrss requested a review from ReimarBauer February 24, 2026 15:34
Copy link
Member

@ReimarBauer ReimarBauer left a comment

Choose a reason for hiding this comment

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

I ask Thomas or Radomir to merge the open PR next days.

I am not sure if we have the same problems xstatic-py/xstatic-font-awesome#4 (comment) with pinning setuptools.

For 10.2.1_3 I did that.

@matrss matrss force-pushed the deduplicate-dependencies branch from bd102bd to 72c7ef6 Compare February 25, 2026 08:13
@ReimarBauer
Copy link
Member

ReimarBauer commented Feb 25, 2026

looks like https://coveralls.io/ has a problem

https://status.coveralls.io/

eventlet = ">0.30.2"
flake8 = "*"
flake8-builtins = "*"
gitpython = "*"
Copy link
Member

Choose a reason for hiding this comment

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

We need the test suite and a manual test on windows for checking

Or do you see that it comes in by another package? Aka What was the reason to remove it?

Copy link
Member

Choose a reason for hiding this comment

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

ignore, I've overseen it in the list

Copy link
Member

@ReimarBauer ReimarBauer left a comment

Choose a reason for hiding this comment

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

Tests succeeded only transmission to coveralls shows an error. This was fixed meanwhile.

@ReimarBauer ReimarBauer merged commit f643aa5 into Open-MSS:develop Feb 27, 2026
7 of 27 checks passed
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.

Duplicate dependencies in pixi.toml

2 participants