Skip to content

bumping ts rules#17760

Merged
diemol merged 3 commits into
trunkfrom
bump-rules_ts_ext
Jul 10, 2026
Merged

bumping ts rules#17760
diemol merged 3 commits into
trunkfrom
bump-rules_ts_ext

Conversation

@diemol

@diemol diemol commented Jul 10, 2026

Copy link
Copy Markdown
Member

💥 What does this PR do?

This pull request updates the TypeScript version used by the project to ensure compatibility with newer features and bug fixes.

🔧 Implementation Notes

🤖 AI assistance

  • No substantial AI assistance used
  • AI assisted (complete below)
    • Tool(s):
    • What was generated:
    • I reviewed all AI output and can explain the change

💡 Additional Considerations

🔄 Types of changes

  • Cleanup (formatting, renaming)
  • Bug fix (backwards compatible)
  • New feature (non-breaking change which adds functionality and tests!)
  • Breaking change (fix or feature that would cause existing functionality to change)

@selenium-ci selenium-ci added the B-build Includes scripting, bazel and CI integrations label Jul 10, 2026
@qodo-code-review

Copy link
Copy Markdown
Contributor

PR Summary by Qodo

Bump TypeScript version to 5.8.3 in rules_ts_ext

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Update the TypeScript toolchain version used by Bazel rules_ts_ext.
• Align builds with newer TypeScript features, fixes, and ecosystem expectations.
High-Level Assessment

The direct version pin bump is the simplest and most appropriate approach for a targeted TypeScript upgrade. The main follow-up is outside this PR: validate CI/builds and consider documenting any required tsconfig or rules_ts compatibility adjustments if failures appear.

Files changed (1) +1 / -1

Other (1) +1 / -1
MODULE.bazelBump TypeScript version used by rules_ts_ext +1/-1

Bump TypeScript version used by rules_ts_ext

• Updates the pinned TypeScript version for the Bazel rules_ts_ext extension from 4.9.5 to 5.8.3, changing which TypeScript compiler is fetched/used during builds.

MODULE.bazel

@qodo-code-review

qodo-code-review Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@qodo-code-review

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit 77ab5ae

@qodo-code-review

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit 08a2510

@diemol diemol merged commit a6a14b4 into trunk Jul 10, 2026
63 checks passed
@diemol diemol deleted the bump-rules_ts_ext branch July 10, 2026 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-build Includes scripting, bazel and CI integrations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants