Skip to content

Add array of accepted synonym packages for sheetify - #165

Merged
emilbayes merged 4 commits into
stackcss:masterfrom
bcomnes:add-other-views
Jun 24, 2019
Merged

Add array of accepted synonym packages for sheetify#165
emilbayes merged 4 commits into
stackcss:masterfrom
bcomnes:add-other-views

Conversation

@bcomnes

Copy link
Copy Markdown
Contributor

I'm looking to export sheetify from hui in order to enforce the same version of the tool across a codebase built on that component library. This PR attempts to add an array similar to what is found in hyperhtml which would enable the ability to use package synonyms.

@bcomnes

Copy link
Copy Markdown
ContributorAuthor

Ping @emilbayes@goto-bus-stop perhaps you would have thoughts on this.

Comment threadtransform.js Outdated
Comment threadtransform.js Outdated
@bcomnes

Copy link
Copy Markdown
ContributorAuthor

@emilbayes ok updated. wdyt?

Array.prototype.includes is not defined in older nodes
@bcomnes

Copy link
Copy Markdown
ContributorAuthor

Looks like includes would be a major version bump. Fine with me.

Comment threadtransform.js Outdated
@bcomnesbcomnes mentioned this pull request Jun 24, 2019
@emilbayes
emilbayes merged commit 6675506 into stackcss:masterJun 24, 2019
@bcomnes
bcomnes deleted the add-other-views branch June 24, 2019 12:02
@bcomnes

Copy link
Copy Markdown
ContributorAuthor

Tests pass up till node 6, so the major version bump should note that.

@bcomnes

Copy link
Copy Markdown
ContributorAuthor

I'm going to release an 8.0.0-beta0 today and ensure this works properly in the wild. After that, I will release the 8.0.0 with a CHANGELOG/release page.

@bcomnes

Copy link
Copy Markdown
ContributorAuthor

Released in 8.0.0-beta.0. Install with npm I sheetify@beta.

Sign up for freeto 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.

3 participants

@bcomnes@emilbayes@goto-bus-stop