deluan / contentful-migrate

🐎 Schema migration tooling for Contentful, with state management
https://www.contentful.com/blog/2018/09/13/content-model-changes-scale-telus-cms-as-code/
MIT License
60 stars 37 forks source link

⬆️ Update contentful dependencies #196

Closed OriginalEXE closed 3 years ago

OriginalEXE commented 3 years ago

Hello,

This PR updates the contentful dependencies (mainly contentful-migration). There was a breaking change in v3.0.0 so I adjusted run.js to take that into account.

Let me know if anything else is needed, all tests are passing.

deluan commented 3 years ago

Hey @OriginalEXE , thanks for the PR! I found an issue with flakiness in the integration tests, but don't want to commit before merging this, so can you please fix the conflicts?

OriginalEXE commented 3 years ago

Hey @deluan, which conflicts are we talking about? I thought I had already resolved them 🤔 Don't see anything in the GitHub UI

michielmetcake commented 3 years ago

Probably needs a version bump?

deluan commented 3 years ago

These conflicts:

Screen Shot 2021-03-22 at 9 55 32 AM

But looks like I can Squash and merge. Will try that.

deluan commented 3 years ago

Great! Will fix the flakiness and hopefully we will be able to release a new version tonight! Thanks for this!

OriginalEXE commented 3 years ago

Ah sorry, did not see that, showed it as merge-able to me 🤷🏻

Thanks for taking care of it!