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 to ember 3.20 LTS #385

Conversation

patocallaghan
Copy link
Contributor

@patocallaghan patocallaghan commented Nov 6, 2020

Since 5.0 beta is introducing breaking changes we may as well upgrade to the latest 3.20 LTS and drop Node 8.

  • Updates to 3.20 LTS
  • Updates other dependencies e.g. ember-copy, git-repo-info
  • Fixes lint errors (from upgrade) and uses new @ember-data/.. import syntax

/cc @jakesjews

@patocallaghan patocallaghan force-pushed the patoc/ember-cli-upgrade branch 6 times, most recently from 24995d0 to 557845c Compare November 7, 2020 23:04
@patocallaghan patocallaghan force-pushed the patoc/ember-cli-upgrade branch from 557845c to 3e889ee Compare November 7, 2020 23:20
@patocallaghan patocallaghan force-pushed the patoc/ember-cli-upgrade branch from 3e889ee to 6eba4bb Compare November 7, 2020 23:24
@patocallaghan patocallaghan marked this pull request as ready for review November 7, 2020 23:40
@jakesjews
Copy link
Contributor

Hey @patocallaghan I'm good to merge this but I still have a fair amount of CI repositories at work I gotta find a way to convert to two-factor authentication before I can rejoin this org. Another option will be if it gets moved to ember-adopted-addons then someone else can merge it.

@patocallaghan
Copy link
Contributor Author

@jakesjews is the blocker here that you need 2FA on your github account to get access to this repo now, but you don't have 2FA enabled on your work repos?

@jakesjews
Copy link
Contributor

jakesjews commented Nov 9, 2020

we have some small internal stuff using my account and we're pretty understaffed at the moment so its hard to justify to management to take time out to check them all

@patocallaghan
Copy link
Contributor Author

@jakesjews since I've commit access now and you were happy with this PR I'm going to merge

@patocallaghan patocallaghan merged commit 049946f into adopted-ember-addons:master Nov 10, 2020
@patocallaghan patocallaghan deleted the patoc/ember-cli-upgrade branch November 10, 2020 22:48
@jakesjews
Copy link
Contributor

Sounds good!

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