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

Kick off 2.0-beta #321

Merged
merged 13 commits into from
Feb 11, 2021
Merged

Kick off 2.0-beta #321

merged 13 commits into from
Feb 11, 2021

Conversation

mixonic
Copy link
Contributor

@mixonic mixonic commented Aug 10, 2020

Per Addepar/ember-table#819 we're pursuing a 3.0 release for ember-table which drops IE11 support and Ember pre-2.8. As ember-table is likely the most active contributor of this library, I would also like to release a 2.0 of vertical collection dropping support for IE11 and Ember < 2.8.

This branch should kick that off.

@mixonic
Copy link
Contributor Author

mixonic commented Aug 11, 2020

Annoyingly although ember-table supports back to Ember 2.4 (back to 1.11 on master!) this library is only tested in Ember 3.x and does not pass on 2.x.

@RobbieTheWagner
Copy link
Member

This seems fine so far. If we're dropping IE11, maybe we should also drop Ember 2.x?

@mixonic
Copy link
Contributor Author

mixonic commented Aug 11, 2020

Addepar relies on 2.x support both for ember-table and various internal component implementations. Ember-table itself is probably the single largest consumer of this library, and since we can't drop Ember 2.x from ember-table yet we should not drop it from here. In fact I would like to add it to the test suite.

Quickly brainstorming, the advantage to dropping 2.x afaict would be a more stable component manager API which may allow an implementation here to operate at a lower level, or at least at a low level on a stable, public API instead of intimate API. But we could probably also use those APIs progressively during a vertical-collection 2.x cycle and drop weight at 3.x.

@apellerano-pw
Copy link

Is there any plans to integrate the horizontal-collection / virtual-collection pattern proposed in #269 into the next major release?

@knownasilya
Copy link
Contributor

knownasilya commented Dec 16, 2020

I'd love to at least get an Ember 3 and IE11 release, since Ember still supports IE11.

@RobbieTheWagner
Copy link
Member

I don't really know much about the internals of this repo, but happy to add anyone as maintainers if they want to help get this over the finish line!

@mixonic
Copy link
Contributor Author

mixonic commented Feb 7, 2021

In alignment with a proposal at Addepar/ember-table#819 (comment), I've opened a PR at #325 moving the minimum supported version of Ember for the 2.0 branch to 2.8.

mixonic and others added 2 commits February 6, 2021 22:53
@twokul twokul marked this pull request as ready for review February 9, 2021 17:38
@mixonic
Copy link
Contributor Author

mixonic commented Feb 9, 2021

@twokul and I discussed and we think the last item here is the ember-raf-scheduler dependency getting a bump.

@twokul
Copy link
Contributor

twokul commented Feb 9, 2021

As per discussion, #328. I believe after it is merged, we can safely merge this PR and proceed with the release of 2.0.

.travis.yml Outdated Show resolved Hide resolved
tests/dummy/config/targets.js Show resolved Hide resolved
@twokul twokul force-pushed the 2.0-beta branch 2 times, most recently from 05febe3 to 11b2ccd Compare February 9, 2021 21:15
@@ -1,6 +1,6 @@
{
"name": "@html-next/vertical-collection",
"version": "1.0.0",
"version": "2.0.0-0",
Copy link
Member

Choose a reason for hiding this comment

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

Should we label this as a beta?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This format of version number is commonly used by NPM packages to identify a pre-release. The published package on NPM is tagged next, which is basically equivalent to beta (only the latest tag on npm is for stable packages).

You prefer the -beta style versioning? Or were you referring to the npm tag?

Copy link
Member

Choose a reason for hiding this comment

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

I'm just used to seeing -beta on betas.

@mixonic
Copy link
Contributor Author

mixonic commented Feb 11, 2021

I've integrated this into ember-table and into a large codebase. I think we're good to land it and cut 2.0!

@twokul twokul merged commit 612093d into master Feb 11, 2021
@twokul twokul deleted the 2.0-beta branch February 11, 2021 14:35
lan0 pushed a commit to lan0/vertical-collection that referenced this pull request Mar 12, 2022
Co-authored-by: Alex Navasardyan <[email protected]>
Co-authored-by: Alex Navasardyan <[email protected]>
Co-authored-by: Robert Wagner <[email protected]>
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.

5 participants