Skip to content

Conversation

@samayer12
Copy link
Contributor

Description

Update peer deps

End to End Test:
(See Pepr Excellent Examples)

Related Issue

Regular maintenance.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@socket-security
Copy link

socket-security bot commented Oct 1, 2025

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​typescript-eslint/​parser@​8.42.0 ⏵ 8.45.0100 +110071 +397 +1100
Updatedesbuild@​0.25.9 ⏵ 0.25.10921007391100
Updated@​typescript-eslint/​eslint-plugin@​8.42.0 ⏵ 8.45.099 +110080 +297 +1100
Updatedtypescript@​5.8.3 ⏵ 5.9.3100 +110090 +110090
Updateduuid@​11.1.0 ⏵ 13.0.010010010091100
Updatedeslint@​9.34.0 ⏵ 9.36.097 +110010094 -1100

View full report

@codecov
Copy link

codecov bot commented Oct 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.43%. Comparing base (6538518) to head (89e3826).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2663   +/-   ##
=======================================
  Coverage   84.43%   84.43%           
=======================================
  Files          95       95           
  Lines        6509     6509           
  Branches      782      782           
=======================================
  Hits         5496     5496           
  Misses       1001     1001           
  Partials       12       12           
Files with missing lines Coverage Δ
src/lib/filter/adjudicators/binding.ts 100.00% <100.00%> (ø)
🚀 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.

@samayer12
Copy link
Contributor Author

The additional work on this PR is related to microsoft/TypeScript#61668 which shipped as part of TypeScript 5.9.0.

Copy link
Contributor

@cmwylie19 cmwylie19 left a comment

Choose a reason for hiding this comment

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

This PR makes sense. We need to update our peer deps and it seems that the PEXEX are failing due to a TypeScript version conflict. We will force merge this and then test again

@github-project-automation github-project-automation bot moved this to 👀 In review in Pepr Project Board Oct 1, 2025
@cmwylie19 cmwylie19 marked this pull request as ready for review October 1, 2025 19:50
@cmwylie19 cmwylie19 requested a review from a team as a code owner October 1, 2025 19:50
@cmwylie19 cmwylie19 merged commit 7452251 into main Oct 1, 2025
23 of 87 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Pepr Project Board Oct 1, 2025
@cmwylie19 cmwylie19 deleted the hotfix-peer-deps branch October 1, 2025 19:50
cmwylie19 pushed a commit that referenced this pull request Oct 1, 2025
…tion (#2665)

## Description

This PR fixes an issue with our peer dep upgrade.


End to End Test:  <!-- if applicable -->  
(See [Pepr Excellent
Examples](https://github.com/defenseunicorns/pepr-excellent-examples))

## Related Issue

Relates to #2663 

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging
- [ ] Unit, Integration,
[docs](https://github.com/defenseunicorns/pepr/tree/main/docs),
[adr](https://github.com/defenseunicorns/pepr/tree/main/adr) added or
updated as needed
- [ ] [Contributor Guide
Steps](https://docs.pepr.dev/main/contribute/#submitting-a-pull-request)
followed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants