chore(deps): patch and minor sweep - #648
Merged
Merged
Conversation
Moves 30 catalog entries to the newest release within their current major, skipping anything younger than the 7 day minimumReleaseAge. Notable jumps: eslint 10.4.1 -> 10.8.0, @nuxtjs/sitemap 8.0.15 -> 8.3.0, @nuxt/content 3.14.0 -> 3.15.2, vue-router 5.0.7 -> 5.2.0, vue 3.5.34 -> 3.5.40, stylelint 17.13.0 -> 17.14.1, @playwright/test 1.60.0 -> 1.62.0. Deliberately left out: - satori and braintree-web, which get their own PRs: satori renders the OG images and wants a visual diff, braintree-web sits on the payment path. - nuxt, vite and their plugins, ui-library, and the web3 stack, which are their own batches. - @vueuse/* 14.4.0, published 6 days ago, so minimumReleaseAge rejects it. Eligible tomorrow. - typescript, @rotki/eslint-config, pinia, @unhead/vue, swiper, bumpp and better-sqlite3 13, all majors being held back. The stylelint bump does not clear the transitive postcss 8.5.15 left over from the security batch after all. stylelint itself moved off it, but stylelint-order 8.1.1 declares its own `postcss: ^8.5.8`, 8.1.1 is the newest release, and pnpm will not re-resolve a range that is already satisfied. Clearing it needs an override, so it stays deferred. typecheck, lint (68 pre-existing warnings, 0 errors), 476 tests and generate (530 routes, 155 OG images) all pass.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #648 +/- ##
=======================================
Coverage 28.42% 28.42%
=======================================
Files 390 390
Lines 12232 12232
Branches 1965 1965
=======================================
Hits 3477 3477
Misses 8656 8656
Partials 99 99
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Batch 3 of the renovate dashboard (#169): the patch and minor sweep. 30 catalog entries move to the newest release within their current major.
Targets were re-resolved from npm rather than taken from the dashboard, and filtered to releases older than the repo's 7 day
minimumReleaseAge, so nothing here is a version pnpm would reject.Notable jumps
The rest are patches: vue-tsc, @vitejs/plugin-vue, vitest, @vitest/coverage-v8, @vue/test-utils, rollup, @rotki/eslint-plugin, autoprefixer, happy-dom, lint-staged, msw, npm-run-all2, plainfp, tsdown, @types/node, @fontsource/roboto, better-sqlite3, @nuxt/test-utils, @commitlint/cli, @commitlint/config-conventional.
Deliberately excluded
minimumReleaseAge. Eligible tomorrow.Correction to an earlier claim
In #645 I said the leftover transitive postcss 8.5.15 would be cleared by bumping stylelint in this sweep. That turned out to be wrong, and it is not fixed here. stylelint 17.14.1 did move itself off (it requires
^8.5.16), butstylelint-order@8.1.1declares its ownpostcss: ^8.5.8, 8.1.1 is the newest release, and pnpm will not re-resolve a range that is already satisfied.pnpm update -r --depth Infinity postcssdoes not budge it.Clearing it needs a
pnpm.overridesentry, which felt like the wrong thing to slip into a sweep. Still dev-only linting of first-party CSS, so the advisory's attacker-controlledsourceMappingURLprecondition does not apply. Happy to add the override in a follow-up if you'd prefer it gone.Verification
typecheckclean.lintclean: 68 warnings, 0 errors, byte-identical to before despite eslint 10.4→10.8 and @rotki/eslint-plugin 1.4→1.5.generatebuilds 530 routes and 155 OG images (144 integration, 6 comparison, 5 feature), unchanged. Sitemap still emits 170 URLs across 181 pages.robots.txtis absent from the static output, but that is expected here rather than a regression from the @nuxtjs/robots bump: the Go backend serves it in production viarobotsTxtHandler(backend/internal/api/routing/routes.go), and Nuxt only handles it in dev.