build(deps): bump brace-expansion, test-exclude and typescript-eslint in /packages/http-client-python - #11322
Conversation
|
Azure Pipelines: Successfully started running 1 pipeline(s). 1 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
commit: |
Bumps [brace-expansion](https://github.com/juliangruber/brace-expansion) to 5.0.7 and updates ancestor dependencies [brace-expansion](https://github.com/juliangruber/brace-expansion), [test-exclude](https://github.com/istanbuljs/test-exclude) and [typescript-eslint](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/typescript-eslint). These dependencies need to be updated together. Updates `brace-expansion` from 5.0.5 to 5.0.7 - [Release notes](https://github.com/juliangruber/brace-expansion/releases) - [Commits](juliangruber/brace-expansion@v5.0.5...v5.0.7) Updates `test-exclude` from 7.0.1 to 7.0.2 - [Release notes](https://github.com/istanbuljs/test-exclude/releases) - [Changelog](https://github.com/istanbuljs/test-exclude/blob/master/CHANGELOG.md) - [Commits](istanbuljs/test-exclude@test-exclude-v7.0.1...test-exclude-v7.0.2) Updates `typescript-eslint` from 8.55.0 to 8.65.0 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-eslint/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.65.0/packages/typescript-eslint) --- updated-dependencies: - dependency-name: brace-expansion dependency-version: 5.0.7 dependency-type: indirect - dependency-name: test-exclude dependency-version: 7.0.2 dependency-type: indirect - dependency-name: typescript-eslint dependency-version: 8.65.0 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
f1fa7a5 to
f6f9c52
Compare
Python emitter diffBaseline No changes to generated output. Rendered diff: inline on the run summary, or the emitter-diff-html artifact. Informational check (eng/emitter-diff); does not block the PR. |
…-python/multi-8e14e314c6
|
Copilot why is ci failing |
|
Copilot the linux builds are looking for @eslint/js |
|
Copilot I think we need to add @eslint/js as a dependency |
Co-authored-by: l0lawrence <100643745+l0lawrence@users.noreply.github.com>
Added |
…-python/multi-8e14e314c6
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: d9d0e1a1-ea80-4398-87be-49658388a53d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: d9d0e1a1-ea80-4398-87be-49658388a53d
|
Hi @Dependabot (@dependabot)[bot]. Your PR has had no update for 14 days and it is marked as a stale PR. If it is not updated within 14 additional days, the PR will automatically be closed. If you want to refresh the PR, please remove the |
…-python/multi-8e14e314c6
There was a problem hiding this comment.
🟡 Changes recommended
The PR metadata doesn’t match the actual changes (typescript-eslint removal/oxlint adoption) and the lockfile introduces ms-feed-*.pkgs.visualstudio.com resolved URLs that may break installs outside that feed.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
Files not reviewed (1)
- packages/http-client-python/package-lock.json: Generated file
- Files reviewed: 4/5 changed files
- Comments generated: 2
- Review effort level: Lite
…rn/packages/http-client-python/multi-8e14e314c6 # Conflicts: # packages/http-client-python/eng/scripts/ci/config/eslint-ci.config.mjs # packages/http-client-python/package-lock.json
There was a problem hiding this comment.
🟡 Changes recommended
The PR description/title doesn’t match the actual changes (tooling migration + extra dependency bumps) and the dependency changes for a standalone package should include an appropriate .chronus/changes/* entry.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
Suppressed comments (2)
packages/http-client-python/package.json:105
- PR description/title focus on brace-expansion/test-exclude/typescript-eslint, but this change also bumps direct dependencies js-yaml and tsx; please update the PR description (or split) so reviewers/release notes accurately reflect the additional dependency updates.
"dependencies": {
"js-yaml": "^4.3.1",
"marked": "^15.0.6",
"pyodide": "0.26.2",
"semver": "~7.6.2",
"tsx": "^4.23.13"
},
packages/http-client-python/package.json:133
- PR description says typescript-eslint is being bumped, but typescript-eslint is removed from devDependencies and oxlint is added instead; please clarify whether this PR is intended to migrate linting tooling or just bump typescript-eslint.
"@types/node": "~25.0.2",
"@types/semver": "7.5.8",
"c8": "^10.1.3",
"oxlint": "^1.69.0",
"picocolors": "~1.1.1",
"rimraf": "~6.1.2",
"typescript": "~5.9.2",
"vitest": "^4.0.15",
- Files reviewed: 5/6 changed files
- Comments generated: 4
- Review effort level: Lite
Bumps brace-expansion to 5.0.7 and updates ancestor dependencies brace-expansion, test-exclude and typescript-eslint. These dependencies need to be updated together.
Updates
brace-expansionfrom 5.0.5 to 5.0.7Commits
b735ba65.0.7c7e33ecMerge commit from forkfa1786aBump tar from 7.5.11 to 7.5.16 (#116)c16f1ecfix(package.json): use git+https instead of git+ssh for repository URL (#115)196bbccBump ip-address from 10.1.0 to 10.2.0 (#104)eb18dd6Update ci.yml (#107)a39928dBump minimatch (#106)7950ae3ci: correct test workflow (#105)46317b55.0.6c0b095bMerge commit from forkUpdates
test-excludefrom 7.0.1 to 7.0.2Release notes
Sourced from test-exclude's releases.
Changelog
Sourced from test-exclude's changelog.
Commits
80dde47chore(master): release test-exclude 7.0.2 (#71)744cb01fix(deps): update minimatch to 10 (#70)Updates
typescript-eslintfrom 8.55.0 to 8.65.0Release notes
Sourced from typescript-eslint's releases.
... (truncated)
Changelog
Sourced from typescript-eslint's changelog.
... (truncated)
Commits
63ba81bchore(release): publish 8.65.0eaf4576feat: add warning when TS 7 is detected (#12529)0d06406chore: add attw validation to repo (#12437)c2386e4chore(deps): update dependency prettier to v3.9.5 (#12486)414d9abchore(release): publish 8.64.0290cf6cchore(release): publish 8.63.03ea32f4chore(release): publish 8.62.154e2857chore(release): publish 8.62.081e4c26feat: remove redundant package.json "files" (#12444)aaad718chore(release): publish 8.61.1