Skip to content

Conversation

jamescdavis
Copy link
Member

@jamescdavis jamescdavis commented May 31, 2019

Purpose

Upgrade dependencies that don't require any code or configuration change.

Summary of Changes

Side Effects

Hope not!

Feature Flags

n/a

QA Notes

n/a

Ticket

n/a

Reviewer Checklist

  • meets requirements
  • easy to understand
  • DRY
  • testable and includes test(s)
  • changes described in CHANGELOG.md

exceptions:
* ember-data-factory-guy - 3.8.1+ requires pretender 3.0+, which conficts with ember-cli-mirage < 1.0
* typescript - 3.5+ "fixes" an issue with property initialization that prevents use of defaultTo
Copy link
Contributor

@brianjgeiger brianjgeiger left a comment

Choose a reason for hiding this comment

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

Should be great if tests and percy passes.

@jamescdavis
Copy link
Member Author

Tests passed in my branch, but good idea to put this through Percy.

@brianjgeiger
Copy link
Contributor

It's almost as if that were one of the specific use cases for which we added Percy. ;)

@jamescdavis
Copy link
Member Author

Closing in favor of #679 so we get a Percy build.

@jamescdavis jamescdavis deleted the upgrade_dependencies branch June 9, 2019 23: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.

2 participants