Skip to content

fix(deps): update is-path-inside to v4#5757

Closed
Copilot wants to merge 5 commits intomainfrom
copilot/sub-pr-5748
Closed

fix(deps): update is-path-inside to v4#5757
Copilot wants to merge 5 commits intomainfrom
copilot/sub-pr-5748

Conversation

Copy link
Contributor

Copilot AI commented Feb 21, 2026

  • Merged main (including latest workflow changes)
  • Kept is-path-inside ^4.0.0 and .default fix in watch-run.js
  • Resolved package-lock.json conflict by regenerating with npm 10
  • Validated with lint, tsc, and unit tests (1100 passing)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 21, 2026

CLA Not Signed

Co-authored-by: mark-wiemer <7833360+mark-wiemer@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix dependency update for is-path-inside to v4 fix: handle ESM-only is-path-inside v4 in CommonJS watch-run Feb 21, 2026
Copilot AI requested a review from mark-wiemer February 21, 2026 03:27
@mark-wiemer
Copy link
Member

@copilot change this PR to merge into main. Always use npm 10.

@mark-wiemer mark-wiemer changed the base branch from renovate/is-path-inside-4.x to main February 21, 2026 03:40
@mark-wiemer mark-wiemer marked this pull request as ready for review February 21, 2026 03:41
Co-authored-by: mark-wiemer <7833360+mark-wiemer@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 21, 2026

@copilot change this PR to merge into main. Always use npm 10.

Done in commit 279c12d. Merged all current main changes into this branch, updated is-path-inside to ^4.0.0, fixed the .default interop in watch-run.js, and regenerated package-lock.json with npm 10. The PR can now be retargeted to main.

Copilot AI changed the title fix: handle ESM-only is-path-inside v4 in CommonJS watch-run fix(deps): update is-path-inside to v4 Feb 21, 2026
@mark-wiemer
Copy link
Member

@copilot resolve conflicts, use npm 10

Co-authored-by: mark-wiemer <7833360+mark-wiemer@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 21, 2026

@copilot resolve conflicts, use npm 10

Resolved in commit 5317d7a. Merged the latest main (new workflow changes), resolved the package-lock.json conflict by regenerating it with npm 10, and kept the is-path-inside v4 + .default fixes intact.

@mark-wiemer
Copy link
Member

closing, gonna do these manually one at a time

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.

2 participants