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 app to Ember 3.22 #206

Merged
merged 8 commits into from
Oct 22, 2020
Merged

Upgrade app to Ember 3.22 #206

merged 8 commits into from
Oct 22, 2020

Conversation

maxwondercorn
Copy link
Contributor

@maxwondercorn maxwondercorn commented Oct 17, 2020

Upgrade Ember per #197

  • Update lint/test-related packages
  • Update remark-related packages
  • Update Ember addons
  • Update Ember-related packages (packages installed by default)
  • Update Ember with ember-cli-update

Closes #197

@maxwondercorn
Copy link
Contributor Author

The Alex linting issues were also present on master

@ijlee2
Copy link
Member

ijlee2 commented Oct 17, 2020

Hi, @maxwondercorn. Thanks for working on upgrading the dependencies to their latest version.

When lint errors happen due to a package upgrade, I'd recommend adding the lint rule as an exception, as demonstrated in ember-learn/ember-website@9999b79. I can create a follow-up issue to fix the lint errors separately.

It looks you can edit .alexrc file. Can you try adding retext-equality to the list of allowed rules, see if this will help Travis CI to pass?

@ijlee2 ijlee2 self-requested a review October 17, 2020 13:45
@maxwondercorn
Copy link
Contributor Author

I am going to add alex disable comments for each error in the markdown files. I think it's better to be specific than globally disabling them in .alexrc

There must have some type of regression. These were not flagged the last time I did a PR. Some of the flagged words are relatively recent additions, but some have been in the guides for a couple years.

@ijlee2
Copy link
Member

ijlee2 commented Oct 17, 2020

Per file sounds good with me, thanks. I think ember-cli just released 3.22 version. If you'd like to upgrade the app to 3.22 instead, feel free to.

@ijlee2
Copy link
Member

ijlee2 commented Oct 17, 2020

@maxwondercorn Looks like the Netlify deploy build failed due to this error:

12:21:32 PM: Build Error (broccoli-persistent-filter:Babel > [Babel: @ember-data/adapter]) in @ember-data/adapter/rest.js
12:21:32 PM: /opt/build/repo/@ember-data/adapter/rest.js: Imported DEPRECATE_NAJAX from @ember-data/private-build-infra/deprecations which is not a supported flag.

I think this is the related pull request: emberjs/data#7230 . It seems to suggest installing ember-fetch to remove the deprecation, but we already have the latest package? Hmm...

@ijlee2
Copy link
Member

ijlee2 commented Oct 17, 2020

According to a Discord conversation, it seems removing node_modules will help fix the error. I will clear the cache and restart the build.

@ijlee2
Copy link
Member

ijlee2 commented Oct 17, 2020

Yep, clearing the cache worked. ✨

@maxwondercorn
Copy link
Contributor Author

I did that locally and if fixed the issue but on the Netilfy build... Glad it's now working!!

@maxwondercorn
Copy link
Contributor Author

@ijlee2 all upgrade steps for the 3.21 bump are completed

@ijlee2
Copy link
Member

ijlee2 commented Oct 21, 2020

@maxwondercorn Thanks! I'll have a look tomorrow.

@ijlee2 ijlee2 changed the title Upgrade app to Ember 3.21 (Part 1 of 2) Upgrade app to Ember 3.22 Oct 22, 2020
Copy link
Member

@ijlee2 ijlee2 left a comment

Choose a reason for hiding this comment

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

@maxwondercorn Thank you for working on upgrading the CLI Guides app to v3.22! Always happy to see Ember apps running on the latest version.

I tested the deployed app by navigating between pages and the app continued to work without error. 💯

@ijlee2 ijlee2 merged commit fe580fb into ember-learn:master Oct 22, 2020
@maxwondercorn maxwondercorn deleted the ember-3.21-upgrade-part1 branch November 2, 2020 00:49
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.

Upgrade app to Ember 3.22 (Part 1 of 2)
2 participants