ember-cp-validations icon indicating copy to clipboard operation
ember-cp-validations copied to clipboard

Release strategy

Open fsmanuel opened this issue 2 years ago • 27 comments

My motivation to help working on the addon is to get rid of the annoying deprecations pointed out in #715 In the update process:

  • #711
  • #712
  • #714
  • #715

I figured out that it's not that simple to get it to work with ember >=4 because the addon and probably the dependencies use the old ember imports and private apis that don't work anymore in ember 4.

That raises the question how we should release new versions to have a happy update path. The latest released version is 4.0.0-beta.12 which was released over a year ago. NPM Versions show that most downloads are for the beta releases.

My suggestion would be to release a version 4.0 that supports ember <=3.28.x and after that work on a new 5.0 release for ember >=4. That would allow us to drop support for ember <=3.28.x in the 5.x branch and make things much easier.

We need to highlight breaking changes from the ember-validators dependency.

I'm happy for more input and opinions.

  • [x] Deprecate volatile option because computed().volatile() is removed:
    • [x] https://github.com/emberjs/ember.js/pull/19670
    • [x] https://github.com/adopted-ember-addons/ember-cp-validations/pull/718
  • [x] Release 4.0.0-beta.13
  • [x] Fix ember 4.0 issues https://github.com/adopted-ember-addons/ember-cp-validations/pull/717
  • [x] #720
  • [x] Release 4.0 for ember <=3.28.x
  • [x] Release 5.0 for ember >=4
  • [x] Replace ember-font-awesome https://github.com/adopted-ember-addons/ember-cp-validations/pull/731
  • [x] Update to ember-cli 4.8 https://github.com/adopted-ember-addons/ember-cp-validations/pull/732
  • [ ] Update to ember-cli 4.12 https://github.com/adopted-ember-addons/ember-cp-validations/pull/736
  • [ ] Replace ember-data imports https://github.com/adopted-ember-addons/ember-cp-validations/issues/740
  • [ ] Enable embroider-safe and embroider-optimized scenarios https://github.com/adopted-ember-addons/ember-cp-validations/issues/739
  • [ ] https://github.com/adopted-ember-addons/ember-cp-validations/issues/722

fsmanuel avatar Jul 07 '22 09:07 fsmanuel

That sounds like a reasonable approach.

knownasilya avatar Jul 07 '22 14:07 knownasilya

This is a crucial dependency of a project I am attempting to update to Ember 4.0. I will gladly help if told what is necessary. Is there a todo list?

andsmedeiros avatar Jul 07 '22 20:07 andsmedeiros

@andsmedeiros not yet. I'll think about it and update the issue.

fsmanuel avatar Jul 08 '22 12:07 fsmanuel

Sounds like a fine approach. A new release (even another beta) would be appreciated

gilest avatar Jul 08 '22 13:07 gilest

Bump to humbly request a beta.13 😅

gilest avatar Jul 11 '22 00:07 gilest

@fsmanuel any progress on this? Can I assist in any way?

andsmedeiros avatar Jul 20 '22 13:07 andsmedeiros

Once again requesting a beta release 😅 Who has release creds? @offirgolan

gilest avatar Jul 31 '22 21:07 gilest

@fsmanuel FYI: https://github.com/qonto/ember-cp-validations/pulls?q=is%3Apr+is%3Aclosed

zeppelin avatar Aug 18 '22 13:08 zeppelin

Hey @zeppelin long time no see! I'll have a look at your PRs and anlredy think it'll help a lot.

fsmanuel avatar Aug 18 '22 13:08 fsmanuel

@fsmanuel thank you for your work. I almost decided to move from ember-cp-validations into other library. You saved us lot of time! Looking forward for 4.0.0-beta.13, 4.x and 5.x versions. We need all of them!

enterpub avatar Aug 23 '22 15:08 enterpub

I had made up my mind to move away from ember-cp-validations too because it did not work with Ember 4. Looking forward to the 4.x and 5.x, thanks @fsmanuel !

JenLyndle avatar Aug 24 '22 21:08 JenLyndle

beta.13 is released. We'll need to replace the deprecated ember-cli-release process, esp since it doesn't work with OTP and the changelog generation seems broken (or maybe the commits aren't properly formatted). Should follow the guidelines in the adopted org.

knownasilya avatar Aug 25 '22 15:08 knownasilya

@knownasilya Thanks a lot.

To all: Would be great to get some help with the release and guidelines as I'm not familiar setting up all that tools.

fsmanuel avatar Aug 25 '22 16:08 fsmanuel

@gilest @andsmedeiros @enterpub @JenLyndle can you test the 4.0.0-beta.13 release and report bugs or problems if you find any?

fsmanuel avatar Aug 25 '22 16:08 fsmanuel

@fsmanuel I will be able to test next week. We have a huge app with all kinds of validation, so I'll test it heavily. Thank you so much!

enterpub avatar Aug 25 '22 16:08 enterpub

@fsmanuel sure will do! thanks!

JenLyndle avatar Aug 25 '22 17:08 JenLyndle

@fsmanuel thanks a lot! I'll test it against my application and let you know of what comes out of it.

andsmedeiros avatar Aug 25 '22 20:08 andsmedeiros

@knownasilya Seems like there are not bugs or problems with the beta-13 release. I would say we setup the release and changelog process and rerelease master as v4.0 what do you think?

fsmanuel avatar Sep 08 '22 12:09 fsmanuel

@knownasilya do you want to do the 4.0 release?

fsmanuel avatar Sep 11 '22 07:09 fsmanuel

@knownasilya If you give me npm access I can do the release.

fsmanuel avatar Sep 21 '22 13:09 fsmanuel

Released v4.0.0

fsmanuel avatar Sep 21 '22 15:09 fsmanuel

https://github.com/adopted-ember-addons/ember-cp-validations/pull/717 is looking for reviewers.

fsmanuel avatar Sep 21 '22 15:09 fsmanuel

Now that #717 has been merged, are we please able to get a release?

kiwi-josh avatar Oct 23 '22 17:10 kiwi-josh

@kiwi-josh I'll do a release in the next days and let you know.

fsmanuel avatar Oct 25 '22 06:10 fsmanuel

@kiwi-josh released 5.0

fsmanuel avatar Oct 27 '22 15:10 fsmanuel

As I mentioned before we have huge app with lot of different validation rules, sometimes complex (by using inline validator). We are using Ember 3.28 and Ember-cp-validations 4.0.0 for almost 2 month now, no issues. The upgrade to version 4.0.0 was smooth. Thank you

enterpub avatar Mar 06 '23 14:03 enterpub

@enterpub Thanks for the feedback!

fsmanuel avatar Mar 09 '23 17:03 fsmanuel