ci: bump github/codeql-action from 4.35.5 to 4.36.0#206
Conversation
Bundle ReportBundle size has no change ✅ |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #206 +/- ##
=======================================
Coverage 99.40% 99.40%
=======================================
Files 33 33
Lines 1859 1859
Branches 530 530
=======================================
Hits 1848 1848
Misses 9 9
Partials 2 2 Continue to review full report in Codecov by Sentry.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis PR updates GitHub Actions workflows to bump CodeQL action versions from v4.35.5 to v4.36.0. The ChangesCodeQL Action Version Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
485d401 to
3e8cefa
Compare
|
@dependabot rebase |
|
Looks like this PR is already up-to-date with main! If you'd still like to recreate it from scratch, overwriting any edits, you can request |
## Summary
Three sanitizer test files (`test/hardening.test.ts`,
`test/sanitize.test.ts`, `test/utl-attachment-sanitize.test.ts`) embed
invisible/control Unicode codepoints (zero-width spaces, bidi overrides,
word-joiner, BOM, NEL) as **literal bytes** to exercise the sanitizer.
The `editorconfig-checker` action — recently auto-bumped on the
GitHub-hosted runner — now classifies these files as `Wrong character
encoding ("binary" instead of "utf-8")` and fails the **required**
`EditorConfig check`, blocking Dependabot PRs #205–#208.
This replaces the 35 invisible literals with `\uXXXX` escape sequences.
**Runtime codepoints are identical** (the TS parser decodes the
escapes), so the sanitizer assertions are unchanged — **784 tests still
pass**. The files are now pure ASCII.
Visible characters (em/en dashes, arrows, emoji, CJK, accented letters)
are intentionally left as-is — valid UTF-8, never the cause.
## Type of change
- [ ] Bug fix (non-breaking)
- [ ] New feature
- [ ] Breaking change
- [ ] Documentation
- [ ] Refactor / chore
- [x] Test / CI
## Checklist
- [x] Tests pass locally (784/784 vitest)
- [x] No behavior change — escapes decode to identical runtime
codepoints
- [x] No secrets/tokens committed
## Related issues
Unblocks #205, #206, #207, #208 (all gated on the same EditorConfig
check).
|
@dependabot rebase |
3e8cefa to
f9b495f
Compare
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 4.35.5 to 4.36.0. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@9e0d7b8...7211b7c) --- updated-dependencies: - dependency-name: github/codeql-action dependency-version: 4.36.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
f9b495f to
110c8a2
Compare
klodr
left a comment
There was a problem hiding this comment.
Reviewed dependency bump, auto-merge enabled — letting CI + CodeRabbit + Scorecard gate the merge.
Bumps github/codeql-action from 4.35.5 to 4.36.0.
Release notes
Sourced from github/codeql-action's releases.
Changelog
Sourced from github/codeql-action's changelog.
... (truncated)
Commits
7211b7cMerge pull request #3927 from github/update-v4.36.0-ebc2d9e2b7740f2fUpdate changelog for v4.36.0ebc2d9eMerge pull request #3926 from github/update-bundle/codeql-bundle-v2.25.5d1f74b7Add changelog note2dc40ceUpdate default bundle to codeql-bundle-v2.25.58449852Merge pull request #3910 from github/henrymercer/repo-size-diff-check72ac23cUpdate excluded required check listc5297a2Merge pull request #3919 from github/henrymercer/workflow-concurrency8ffeae7CI: Automatically cancel non-generated workflowsf3f52bfRevertgetErrorMessageimport