Skip to content
This repository was archived by the owner on Nov 8, 2024. It is now read-only.

Changes to minim-api-description #18

Merged
merged 3 commits into from
Nov 29, 2018
Merged

Changes to minim-api-description #18

merged 3 commits into from
Nov 29, 2018

Conversation

pksunkara
Copy link
Contributor

@pksunkara pksunkara commented Nov 28, 2018

I used the default github's .gitignore for Node.js

@pksunkara pksunkara requested a review from kylef November 28, 2018 03:16
@@ -9,7 +9,8 @@
"release": "lerna publish",
"release:force": "lerna publish --force-publish",
"test": "lerna exec --scope='*' -- npm run test",
"lint": "lerna exec --scope='*' -- npm run lint"
"lint": "lerna exec --scope='*' -- npm run lint",
"cover": "exit 0"
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 will be implemented later

Copy link
Member

@kylef kylef left a comment

Choose a reason for hiding this comment

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

Before the next release we will need to restore the browser bundle formerly created in the NPM package ("prepublish": "npm run ci && npm run build && npm run web",). Perhaps we can create separate issue to track.

@@ -21,6 +22,6 @@
"mocha": "^5.0.2"
},
"engines": {
"node": ">=4"
"node": ">=6"
Copy link
Member

Choose a reason for hiding this comment

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

The CHANGELOG.md file from the repo seems to be missing, We should mark a breaking change that Node 4 support was removed.

@pksunkara pksunkara changed the title No more babel in minim-api-description Changes to minim-api-description Nov 28, 2018
@pksunkara pksunkara force-pushed the pksunkara/changes branch 2 times, most recently from d34f104 to 91390b2 Compare November 28, 2018 15:53
Copy link
Member

@kylef kylef left a comment

Choose a reason for hiding this comment

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

👍 I just want to see the following change and we're good to merge this. Just so we don't forget about this breaking change before our next release.

  • Add CHANGELOG entry for Master that there is a Breaking change because Node 4 is now unsupported.

@kylef kylef merged commit c9a60d2 into subtree Nov 29, 2018
@kylef kylef deleted the pksunkara/changes branch November 29, 2018 04:19
pksunkara added a commit that referenced this pull request Nov 30, 2018
pksunkara pushed a commit that referenced this pull request Dec 12, 2018
pksunkara added a commit that referenced this pull request Dec 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants