Skip to content

feat(codemods): log what and why files were skipped#10486

Merged
runspired merged 4 commits intowarp-drive-data:mainfrom
BobrImperator:codemod/skipped-file-results
Feb 25, 2026
Merged

feat(codemods): log what and why files were skipped#10486
runspired merged 4 commits intowarp-drive-data:mainfrom
BobrImperator:codemod/skipped-file-results

Conversation

@BobrImperator
Copy link
Contributor

Depends on #10474

  • Adds better heuristics for what files have failed during which phase of the codemod
    The codemod fails in some situations, especially for models that re-export other models or libraries that can't be easily read and analyzed.

E.g.

/Users/bobrimperator-mac/apps/client/app/mixins/datatables/datatable-controller.js
/Users/bobrimperator-mac/apps/client/app/core/base-model.js
/Users/bobrimperator-mac/apps/client/app/models/cron.js
✅ Migration complete!
📊 Processed: 690
⏭️ Skipped: 293 (81 .d.ts files)

@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies labels Feb 24, 2026
@BobrImperator BobrImperator force-pushed the codemod/skipped-file-results branch from 7e55461 to 104b089 Compare February 25, 2026 12:24
@runspired runspired merged commit b3372e5 into warp-drive-data:main Feb 25, 2026
19 of 21 checks passed
@BobrImperator BobrImperator deleted the codemod/skipped-file-results branch March 4, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants