Skip to content

fix: Bump python & javascript dependencies#9827

Open
asheshv wants to merge 3 commits intomasterfrom
fix/bump-python-js-dependencies
Open

fix: Bump python & javascript dependencies#9827
asheshv wants to merge 3 commits intomasterfrom
fix/bump-python-js-dependencies

Conversation

@asheshv
Copy link
Copy Markdown
Contributor

@asheshv asheshv commented Apr 6, 2026

Summary

  • Update Python dependencies: google-auth-oauthlib 1.3.0 → 1.3.1, sphinxcontrib-youtube 1.4.1 → 1.5.0, fixtures 4.3.1 → 4.3.2, and fix missing newlines at EOF
  • Replace deprecated @babel/plugin-proposal-* packages with their @babel/plugin-transform-* equivalents
  • Remove unused @types/classnames dependency

Summary by CodeRabbit

  • Chores
    • Updated pinned dependencies (google-auth-oauthlib, sphinxcontrib-youtube, fixtures)
    • Replaced Babel "proposal" plugins with corresponding "transform" variants across build/config
    • Removed an unused type dependency
    • Extended CI test matrix to include a PostGIS version dimension and align installed PostGIS packages

Update Python dependencies:
- google-auth-oauthlib 1.3.0 → 1.3.1
- sphinxcontrib-youtube 1.4.1 → 1.5.0
- fixtures 4.3.1 → 4.3.2
- Add missing newline at end of requirements files

Update JavaScript dependencies:
- Replace deprecated @babel/plugin-proposal-* packages with
  @babel/plugin-transform-* equivalents
- Remove unused @types/classnames dependency
- Update yarn.lock

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 32c4eace-72f7-4d06-984a-acbcc55302c7

📥 Commits

Reviewing files that changed from the base of the PR and between 0c16add and 4b3ca5d.

📒 Files selected for processing (1)
  • .github/workflows/run-python-tests-epas.yml

Walkthrough

Update dependency pins (Python and tooling), replace Babel proposal plugins with transform equivalents in web build and lint configs, adjust web package.json, and add a PostGIS matrix dimension with parameterized package selection in GitHub Actions.

Changes

Cohort / File(s) Summary
Python dependency updates
requirements.txt, tools/requirements.txt, web/regression/requirements.txt
Bumped pinned versions: google-auth-oauthlib 1.3.0→1.3.1, sphinxcontrib-youtube 1.4.1→1.5.0, fixtures 4.3.1→4.3.2; added/ensured trailing newlines.
Babel / ESLint / Webpack config changes
web/.eslintrc.js, web/babel.config.json, web/webpack.config.js
Replaced proposal plugins with transform variants: @babel/plugin-proposal-class-properties@babel/plugin-transform-class-properties and proposal object rest/spread → @babel/plugin-transform-object-rest-spread in ESLint, Babel, and webpack loader configs.
Web package.json edits
web/package.json
Swapped proposal plugin packages for transform equivalents in dependencies/devDependencies; removed @types/classnames.
CI workflow matrix change
.github/workflows/run-python-tests-epas.yml
Added postgisver to the test matrix, set default PostGIS 34 for PG 14–17, excluded {pgver:18,postgisver:34}, and explicitly included PostGIS 35 for Postgres 18; Linux install uses edb-as${{ matrix.pgver }}-postgis${{ matrix.postgisver }}.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: bumping both Python and JavaScript dependencies across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bump-python-js-dependencies

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

EPAS 18 ships with edb-as18-postgis35 instead of postgis34.
Add a postgisver matrix variable to support mixed versions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/run-python-tests-epas.yml:
- Around line 36-39: The matrix.include entry does not override the base matrix
so the invalid combination pgver: 18 + postgisver: 34 remains and include items
lack matrix.os; fix by either adding an explicit matrix.exclude entry for
{pgver: 18, postgisver: 34} to remove that invalid combo or by changing the base
lists so postgisver: 34 is not listed for pgver 18, and update every
matrix.include object to include os (so matrix.os is defined for runs-on: ${{
matrix.os }}); reference the matrix keys (matrix.include, pgver, postgisver,
matrix.os) and ensure the exclude or corrected include removes the
edb-as18-postgis34 request.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d813ac47-f376-4708-9dd5-6043aefb5fd1

📥 Commits

Reviewing files that changed from the base of the PR and between 15b3259 and 0c16add.

📒 Files selected for processing (1)
  • .github/workflows/run-python-tests-epas.yml

The previous include entry for pgver 18 with postgisver 35 did not match
any existing matrix combination (postgisver 34 != 35), so GitHub Actions
created a new entry without an os value, causing runs-on to evaluate to ''.

Use exclude to remove pgver 18 / postgisver 34 combos, then include with
explicit os to re-add them with postgisver 35.
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.

1 participant