Skip to content

Update python-matrix-runner#101

Merged
JonatanAntoni merged 1 commit intomainfrom
python-matrix-runner
Apr 22, 2026
Merged

Update python-matrix-runner#101
JonatanAntoni merged 1 commit intomainfrom
python-matrix-runner

Conversation

@JonatanAntoni
Copy link
Copy Markdown
Member

@JonatanAntoni JonatanAntoni requested review from Copilot and jkrech and removed request for Copilot April 22, 2026 09:30
@JonatanAntoni JonatanAntoni marked this pull request as ready for review April 22, 2026 09:30
@JonatanAntoni JonatanAntoni requested a review from Copilot April 22, 2026 09:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Python build dependencies used by Project/build.py (per the referenced Dependabot security alert) and adjusts repository ignore rules for generated artifacts.

Changes:

  • Bump python-matrix-runner dependency from ~=1.0 to ~=1.3.
  • Remove lxml from Project/requirements.txt.
  • Add Project/tmp to .gitignore.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
Project/requirements.txt Updates Python dependency versions for the build.py workflow.
.gitignore Ignores Project/tmp to prevent temporary build artifacts from being committed.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

Test Results

    64 files     64 suites   0s ⏱️
   161 tests   157 ✅   0 💤   4 ❌
10 304 runs  9 472 ✅ 704 💤 128 ❌

For more details on these failures, see this check.

Results for commit e803708.

Copy link
Copy Markdown

@jkrech jkrech left a comment

Choose a reason for hiding this comment

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

This means we do not actually need lxml at all. Great.

@JonatanAntoni
Copy link
Copy Markdown
Member Author

@VladimirUmek, there are failing test cases. May I ask you to check if these are known and expected?

@JonatanAntoni
Copy link
Copy Markdown
Member Author

JonatanAntoni commented Apr 22, 2026

This means we do not actually need lxml at all. Great.

Not exactly, its a transitive dependency of python-matrix-runner. Thus I needed to release a new python-matrix-runner which has all dependencies updated.

@jkrech jkrech requested a review from VladimirUmek April 22, 2026 11:27
@JonatanAntoni JonatanAntoni merged commit 7c3a777 into main Apr 22, 2026
19 of 20 checks passed
@JonatanAntoni JonatanAntoni deleted the python-matrix-runner branch April 22, 2026 12:55
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