Repository navigation
⬆️ Move to pedantry 2.0.2 with pedantry migrate - #18
Merged
Merged
Conversation
Migrated with `pedantry migrate` from v2.0.2 (types: typescript, css). Replaced: the `.pedantry` submodule and `.gitmodules`; `.prettierrc.json5` (now `"prettier": "@dryan/pedantry/prettier"` in package.json); the identical tsconfig options (now `extends: "@dryan/pedantry/tsconfig"`); the identical `.editorconfig` sections (now the managed block); the stale or identical lefthook prettier and eslint commands (now managed); the `.pedantry` entry in the eslint ignores. Preserved: tsconfig `outDir`, `ignoreDeprecations`, `baseUrl` and `paths`; the repo's other lefthook hooks (moved to lefthook-local.yml). New: @dryan/pedantry dev dependency, pedantry.toml, CLAUDE.md, the `@.pedantry/AGENTS.md` import in AGENTS.md, generated .pedantry/AGENTS.md. tsconfig options set to false to preserve old behaviour: none. Hand-fixed: - eslint.config.ts was a full copy of the old pedantry ESLint config; it now spreads `@dryan/pedantry/eslint` and keeps only the repo's `docs/dist/**` and `docs/sentry-init.js(.map)` ignores. The old copy's `projectService` parser options are not carried over. - stylelint.config.mjs (copy of the old pedantry config) now extends `@dryan/pedantry/stylelint` and keeps its `ignoreFiles`. - custom-elements-manifest.config.js was identical to the helper's defaults; it now calls `customElementsManifestConfig()`. - rollup.config.js now builds both configs with `rollupConfig` from `@dryan/pedantry/rollup`, keeping the repo's plugins (json, copy-to-docs), watch settings and `external: []`. `rollup -c` produces byte-identical dist/index.js, docs/dist/index.js and docs/sentry-init.js. - .github/copilot-instructions.md no longer names `.pedantry/tsconfig.json` and `.pedantry/eslint.config.ts`; the section describes the package paths and `pedantry sync`. Checks: pedantry check passes; eslint and stylelint pass. tsc still reports one error in examples.ts (unused variable), as on main; the two eslint.config.ts type errors from the old copy are gone. npm test passes here (it failed in the before run, which had no local build; not caused by this change). Prettier flags the same file as on main (the submodule's copy no longer counts). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
prideraiser
Bot
requested review from
a team and
dryan
and removed request for
a team
October 21, 2026 18:25
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
flags | 7729029 | Oct 21 2026, 06:26 PM |
dryan
approved these changes
Oct 21, 2026
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
1 participant
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.
Migrated with
pedantry migratefrom v2.0.2 (types: typescript, css).Replaced: the
.pedantrysubmodule and.gitmodules;.prettierrc.json5(now"prettier": "@dryan/pedantry/prettier"in package.json); the identical tsconfig options (nowextends: "@dryan/pedantry/tsconfig"); the identical.editorconfigsections (now the managed block); the stale or identical lefthook prettier and eslint commands (now managed); the.pedantryentry in the eslint ignores.Preserved: tsconfig
outDir,ignoreDeprecations,baseUrlandpaths; the repo's other lefthook hooks (moved to lefthook-local.yml).New: @dryan/pedantry dev dependency, pedantry.toml, CLAUDE.md, the
@.pedantry/AGENTS.mdimport in AGENTS.md, generated .pedantry/AGENTS.md. tsconfig options set to false to preserve old behaviour: none.Hand-fixed:
@dryan/pedantry/eslintand keeps only the repo'sdocs/dist/**anddocs/sentry-init.js(.map)ignores. The old copy'sprojectServiceparser options are not carried over.@dryan/pedantry/stylelintand keeps itsignoreFiles.customElementsManifestConfig().rollupConfigfrom@dryan/pedantry/rollup, keeping the repo's plugins (json, copy-to-docs), watch settings andexternal: [].rollup -cproduces byte-identical dist/index.js, docs/dist/index.js and docs/sentry-init.js..pedantry/tsconfig.jsonand.pedantry/eslint.config.ts; the section describes the package paths andpedantry sync.Checks: pedantry check passes; eslint and stylelint pass. tsc still reports one error in examples.ts (unused variable), as on main; the two eslint.config.ts type errors from the old copy are gone. npm test passes here (it failed in the before run, which had no local build; not caused by this change). Prettier flags the same file as on main (the submodule's copy no longer counts).
Co-Authored-By: Claude Sonnet 5.5 noreply@anthropic.com
🤖 Generated with Claude Code