Skip to content

Reviewed aklkv's feat/ember-v2-format#1

Open
ijlee2 wants to merge 6 commits intoaklkv:feat/ember-v2-formatfrom
ijlee2:review-aklkv-ember-v2-format
Open

Reviewed aklkv's feat/ember-v2-format#1
ijlee2 wants to merge 6 commits intoaklkv:feat/ember-v2-formatfrom
ijlee2:review-aklkv-ember-v2-format

Conversation

@ijlee2
Copy link

@ijlee2 ijlee2 commented Feb 10, 2026

What changed?

  • Reviewed JSDocs comments for readability.
  • Split code in index.ts into multiple files.
  • Simplified package configurations.

Further notes:

  • I suspect that config/ember-cli-update.json can be removed.
  • We should combine MIGRATION.md and UPGRADE.md.

@ijlee2 ijlee2 marked this pull request as ready for review February 10, 2026 11:20
@ijlee2 ijlee2 force-pushed the review-aklkv-ember-v2-format branch from c4ca981 to e982da6 Compare February 10, 2026 11:42
@ijlee2 ijlee2 force-pushed the review-aklkv-ember-v2-format branch 2 times, most recently from ee53ae1 to 521a99b Compare February 10, 2026 11:53
@ijlee2 ijlee2 force-pushed the review-aklkv-ember-v2-format branch from 521a99b to 37df98e Compare February 10, 2026 11:54
@aklkv aklkv force-pushed the feat/ember-v2-format branch 3 times, most recently from 03d2cd6 to ed090b2 Compare February 10, 2026 12:35
@aklkv aklkv force-pushed the feat/ember-v2-format branch from ed090b2 to b103a2d Compare March 3, 2026 16:51
@aklkv aklkv force-pushed the feat/ember-v2-format branch from b103a2d to 9d5855e Compare March 17, 2026 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants