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

Drop Node 4 support #744 #839

Merged
merged 3 commits into from
Mar 29, 2018
Merged

Drop Node 4 support #744 #839

merged 3 commits into from
Mar 29, 2018

Conversation

kohgpat
Copy link
Contributor

@kohgpat kohgpat commented Feb 22, 2018

Hello. It's my attempt to fix #744 . Hope I've made everything right.

I've a little question about package-lock.json. After I made changes I installed dependencies again with npm install and got a different package-lock.json and added it to the commit. Is this correct? Just want to be sure package-lock.json behaves like yarn.lock . I'm not using npm 5 currently.

Thank you.

Copy link
Member

@sapegin sapegin left a comment

Choose a reason for hiding this comment

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

Thanks! Looks great, just one tiny remark.

About the lock file: it works the same way as yarn.lock, but there's some issues with npm, so it get updated even when you don't change any dependencies, like in this case. So better to revert it back ;-)

.travis.yml Outdated
@@ -1,12 +1,10 @@
language: node_js
node_js:
- 4
- 6
- 8
- 9
matrix:
Copy link
Member

Choose a reason for hiding this comment

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

We can remove the whole matrix block now.

@sapegin sapegin added this to the 7.0.0 milestone Feb 22, 2018
@kohgpat
Copy link
Contributor Author

kohgpat commented Feb 22, 2018

Updated PR.

@sapegin
Copy link
Member

sapegin commented Feb 22, 2018

Could you remove changes in package-lock.json too?

@kohgpat
Copy link
Contributor Author

kohgpat commented Feb 22, 2018

Oops, I was sure I've revert it already. Sure.

@styleguidist-bot
Copy link
Collaborator

styleguidist-bot commented Feb 22, 2018

Warnings
⚠️

Changes were made to package.json, but not to package-lock.json.

Perhaps you need to run npm install and commit changes in package-lock.json. Make sure you’re using npm 5+.

Generated by 🚫 dangerJS

@kohgpat
Copy link
Contributor Author

kohgpat commented Feb 22, 2018

Fixed.

@kohgpat
Copy link
Contributor Author

kohgpat commented Feb 23, 2018

@sapegin Don't want to bother but is everything ok this time?

@sapegin
Copy link
Member

sapegin commented Feb 23, 2018

YES!!! I've approved the PR and will include it in 7.0.0 release (don't know yet when though ;-)

@sapegin sapegin merged commit d66be96 into styleguidist:master Mar 29, 2018
sapegin added a commit that referenced this pull request Mar 31, 2018
👋 **[Support Styleguidist](https://opencollective.com/styleguidist) on Open Collective** 👋

## Breaking changes

### Node 6 is the lowest supported version

Styleguidist doesn’t support Node 4 anymore.

(#744 #839 by @kohgpat)

### New format of template option

We’re now using [mini-html-webpack-plugin](https://github.com/styleguidist/mini-html-webpack-plugin) and [@vxna/mini-html-webpack-template](https://github.com/vxna/mini-html-webpack-template) instead of [html-webpack-plugin](https://github.com/jantimon/html-webpack-plugin).

This will make things like [adding a favicon](https://react-styleguidist.js.org/docs/cookbook#how-to-add-a-favicon) or fonts from [Google Fonts](https://react-styleguidist.js.org/docs/cookbook#how-to-add-fonts-from-google-fonts) much easier.

If you’re using a custom HTML template, you need to update your style guide config.

```js
// 6.x
module.exports = {
  template: './styleguie/template.html'
}

// 7.x
module.exports = {
  template: {
    // This is just an example, there are many more available options
    favicon: 'https://assets-cdn.github.com/favicon.ico'
  }
}
```

See more [in the docs](https://react-styleguidist.js.org/docs/configuration#template).

(#896 #907 by @sapegin)

### Changed Node API

`server` method now returns an object containing a webpack `Compiler` instance and the Styleguidist server, see examples [in the docs](https://react-styleguidist.js.org/docs/api.html#servercallback).

(#828 by @cmswalker)

## New features

### Webpack 4 support

Webpack 3 is still supported too.

(#857 by @kontrollanten, #900 #901 by @rubenmoya)

### Examples are wrapped in React.Fragment

You don’t need to wrap multiple JSX tags in a div anymore.

```jsx
// 6.x
<div>
  <Button primary>Primary button</Button>
  <Button secondary>Secondary button</Button>
</div>

// 7.x
<Button primary>Primary button</Button>
<Button secondary>Secondary button</Button>
```

(#840 #842 by @tizmagik)

## Bug fixes

* `components` option should accept arrays.
* Do not convert URLs in Markdown to auto links (#793).
* Improved accessibility (#893 by @gergely-nagy).

---

❤️ Huge thanks to @okonet to help with this release, [CI](#904) and [docs](#905) improvements.️ ❤️
@styleguidist-bot
Copy link
Collaborator

🎉 This PR is included in version 7.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Is it time to drop Node 4 support yet?
3 participants