Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade add-on to Ember.js v5.4.0 #305

Merged
merged 8 commits into from Nov 12, 2023
Merged

Conversation

MrChocolatine
Copy link
Member

@MrChocolatine MrChocolatine commented Oct 28, 2023

Breaking

Upgrade add-on to Ember.js v5.4.0 (#305)

  • Ember.js v4.8 or above is now required
  • Ember CLI v4.8 or above is now required
  • Node.js v18 or above is now required

Chore

Fully migrate add-on to TypeScript (#305)

Build

Dependencies updates (#305)

Added

  • @babel/core@^7.23.2
  • @glint/environment-ember-loose@^1.2.1
  • @glint/template@^1.2.1
  • broccoli-asset-rev@^3.0.0
  • concurrently@^8.2.2
  • ember-cli-inject-live-reload@^2.1.0
  • ember-cli-sri@^2.1.1
  • eslint-config-prettier@^9.0.0
  • eslint-plugin-n@^16.2.0
  • eslint-plugin-prettier@^5.0.1
  • prettier@^3.0.3
  • rimraf@^5.0.1
  • ember-source >= 4.0.0 is now a peer dependency

Updated

  • ember-cli-babel@^8.2.0
  • ember-cli-htmlbars@^6.3.0
  • @ember/test-helpers@^3.2.0
  • @typescript-eslint/eslint-plugin@^6.9.1
  • @typescript-eslint/parser@^6.9.1
  • ember-auto-import@^2.6.3
  • ember-cli@~5.4.0
  • ember-cli-dependency-checker@^3.3.2
  • ember-qunit@^8.0.1
  • ember-source@~5.4.0
  • ember-try@^3.0.0
  • eslint@^8.52.0
  • eslint-plugin-ember@^11.11.1
  • eslint-plugin-qunit@^8.0.1
  • qunit@^2.20.0
  • typescript@^5.2.2
  • webpack@^5.89.0

Removed

  • ember-cli-typescript@^5.2.1
  • babel-eslint@^10.1.0
  • ember-cli-typescript-blueprints@^3.0.0
  • ember-disable-prototype-extensions@^1.1.3
  • ember-template-lint@^5.7.2
  • eslint-plugin-node@^11.1.0
  • npm-run-all@^4.1.5

Supersedes and closes #286 .


Below is just extra text to give more context, but it should not be included in our changelog.

Actions taken

  1. Fresh add-on created using the latest version of Ember.js + flag --typescript (using this brings additional changes compared to a simple ember new MyAddon)
  2. Then re-import everything from our add-on
  3. Re-adjust a few things to make it work for our setup

Note to reviewers

When working on this I noticed we had some files that had extra whitespaces:

some content              <= this line has trailing whitespaces
   <= a new line with extra whitespaces too
etc

We are in 2023 we should not encounter such situations anymore, modern tools should take care of this.
Please configure your editor accordingly.

For instance with VSCode you need to enable this setting:

"files.trimTrailingWhitespace": true,

@MrChocolatine MrChocolatine added dependencies Changes that update a dependency file ts-types ci Continuous Integration breaking Breaking changes labels Oct 28, 2023
@MrChocolatine MrChocolatine marked this pull request as ready for review October 28, 2023 22:47
@MrChocolatine MrChocolatine requested a review from a team as a code owner October 28, 2023 22:47
@MrChocolatine MrChocolatine changed the title build: upgrade add-on to Ember.js v5.3.0 Upgrade add-on to Ember.js v5.3.0 Oct 28, 2023
@MrChocolatine MrChocolatine marked this pull request as draft October 28, 2023 23:01
@MrChocolatine MrChocolatine changed the title Upgrade add-on to Ember.js v5.3.0 WIP // Upgrade add-on to Ember.js v5.3.0 Oct 28, 2023
@MrChocolatine MrChocolatine changed the title WIP // Upgrade add-on to Ember.js v5.3.0 Upgrade add-on to Ember.js v5.3.0 Oct 29, 2023
@MrChocolatine MrChocolatine marked this pull request as ready for review October 29, 2023 00:47
@MrChocolatine MrChocolatine force-pushed the upgrade-to-emberjs-5.3.0 branch 2 times, most recently from 1d3bb44 to b82e8dc Compare October 29, 2023 01:00
ndekeister-us
ndekeister-us previously approved these changes Oct 29, 2023
Copy link
Member

@ndekeister-us ndekeister-us left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

Copy link
Member

@ndekeister-us ndekeister-us left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding your last commit running manually prettier, maybe in the future we should add some git hooks or something on CI to prevent merging if there are some prettier errors (would have expected them to be detected via the lint job but 🤷‍♂️)

@MrChocolatine MrChocolatine force-pushed the upgrade-to-emberjs-5.3.0 branch 11 times, most recently from 58412f6 to 1d4c5e0 Compare October 29, 2023 16:32
@MrChocolatine MrChocolatine removed their assignment Oct 30, 2023
@MrChocolatine MrChocolatine marked this pull request as draft October 30, 2023 19:09
@MrChocolatine MrChocolatine changed the title Upgrade add-on to Ember.js v5.3.0 Upgrade add-on to Ember.js v5.4.0 Nov 3, 2023
@MrChocolatine MrChocolatine force-pushed the upgrade-to-emberjs-5.3.0 branch 2 times, most recently from 7eb00a1 to a644eb6 Compare November 3, 2023 23:30
@MrChocolatine MrChocolatine marked this pull request as ready for review November 3, 2023 23:36
@MrChocolatine MrChocolatine dismissed ndekeister-us’s stale review November 4, 2023 13:08

New review requested: I applied another upgrade (v5.3.0 to v5.4.0)

BREAKING CHANGE: upgrade add-on to Ember.js v5.4.0

- Now require Ember.js v4.8 or above
- Now require Ember CLI v4.8 or above
- Now require Node.js v18 or above
There were some warnings when running `npm publish --dry-run`:

```
➜  npm publish --dry-run
npm WARN publish npm auto-corrected some errors in your package.json when publishing.  Please run "npm pkg fix" to address these errors.
npm WARN publish errors corrected:
npm WARN publish "repository" was changed from a string to an object
npm WARN publish "repository.url" was normalized to "git+https://github.com/DazzlingFugu/ember-cli-embedded.git"
npm notice

... logs ...
```
@MrChocolatine
Copy link
Member Author

MrChocolatine commented Nov 11, 2023

PR initially created the 28 October 2023, after 2 weeks I am going to merge it soon.

@MrChocolatine MrChocolatine merged commit e980264 into master Nov 12, 2023
9 checks passed
@MrChocolatine MrChocolatine deleted the upgrade-to-emberjs-5.3.0 branch November 12, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking changes ci Continuous Integration dependencies Changes that update a dependency file documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants