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 Ecto #60

Open
wants to merge 1 commit into
base: bc-new-fixture-api
Choose a base branch
from
Open

Upgrade Ecto #60

wants to merge 1 commit into from

Conversation

cjfreeze
Copy link

@cjfreeze cjfreeze commented Dec 1, 2017

This PR Upgrades Ecto to a more recent version, allowing compatibility with an incoming DockYard API PR. Merging this PR into master would allow the previously mentioned PR to use this library without directly referencing the branch in in its dependency declaration. To accomplish this, a few mix.exs lines were updated, a forgotten field declaration had to be added, and the TravisCI Elixir version had to be updated to 1.5.1. This PR is currently a blocker for https://github.com/DockYard/dockyard-api/pull/232.

@cjfreeze cjfreeze added the WIP label Dec 1, 2017
@cjfreeze cjfreeze force-pushed the cf-upgrade-ecto branch 3 times, most recently from c7a3c69 to 666319c Compare December 1, 2017 16:07
@cjfreeze cjfreeze changed the title Cf upgrade ecto Upgrade Ecto Dec 1, 2017
@cjfreeze cjfreeze added enhancement and removed WIP labels Dec 1, 2017
@cjfreeze cjfreeze requested review from brzpegasus and removed request for brzpegasus December 1, 2017 16:14
@cjfreeze cjfreeze force-pushed the cf-upgrade-ecto branch 7 times, most recently from 9c0e5d3 to 72c3ae9 Compare December 1, 2017 16:54
@cjfreeze cjfreeze changed the base branch from master to bc-new-fixture-api December 1, 2017 16:54
Bumps Ecto dep to ~> 2.2.0 and upgrades ecto where applicable in app.
Adds :elixirc_paths config to mix.exs
Adds a forgotten field type to Order.
Updates TravisCI build version to 1.5.1 to be compatible with Ecto 2.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant