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

[@rollup/plugin-node-resolve] Fix required versions of Node/Rollup in README + include these breaking changes in CHANGELOG #1222

Open
1 of 4 tasks
0xdevalias opened this issue Jul 13, 2022 · 3 comments

Comments

@0xdevalias
Copy link

0xdevalias commented Jul 13, 2022

  • Rollup Plugin Name: @rollup/plugin-node-resolve
  • Rollup Plugin Version:

Documentation Is:

  • Missing
  • Needed
  • Confusing
  • Not Sure?

Please Explain in Detail...

Should the README be updated to align with this then? As currently it says:

This plugin requires an LTS Node version (v8.0.0+) and Rollup v1.20.0+.

Yet package.json suggests that it actually requires Node >= 10.0.0 and peerDependencies of "rollup": "^2.42.0"

Looking at the CHANGELOG, i'm not sure this was marked as a breaking change anywhere either was it?

Originally posted by @0xdevalias in #1023 (comment)

Your Proposal for Changes

The README and CHANGELOG should be kept up to date with breaking changes introduced in package versions, in end-user friendly descriptive language.

@0xdevalias 0xdevalias changed the title [@rollup/plugin-node-resolve] [@rollup/plugin-node-resolve] Fix required versions of Node/Rollup in README + include these breaking changes in CHANGELOG Jul 13, 2022
@0xdevalias
Copy link
Author

0xdevalias commented Jul 13, 2022

It looks like node 10.x was required since e632469 (31 Jul 2020)

https://github.com/rollup/plugins/blob/master/packages/node-resolve/CHANGELOG.md#v900

v9.0.0
Breaking Changes

  • chore: update dependencies (e632469)

It looks like rollup 2.x was required since 3a543df (1 May 2021)

https://github.com/rollup/plugins/blob/master/packages/node-resolve/CHANGELOG.md#v1300

v13.0.0
Breaking Changes


While technically these were both included in the CHANGELOG, they weren't particularly user friendly/obvious in their intent. In contrast, an earlier 'breaking changes' was very clear:

6.0.0

  • Breaking: Minimum compatible Rollup version is 1.20.0
  • Breaking: Minimum supported Node version is 8.0.0

@agilgur5
Copy link

agilgur5 commented Sep 17, 2022

Similar story for #1245, which requires Rollup 2.78.0 now, but doesn't mention this in the CHANGELOG.md entry besides saying it's "breaking" in some way.

The changelog is auto-generated, which encounters some issues where human intervention / readability is required, like in this case.

I was looking to fix these (and the lack of hyperlinks to PRs in the Markdown) at one point in time too, but rather than a simple CHANGELOG.md change, this would require changes to the automated script and more. The script doesn't really have a feature for manual notes, so this would likely require a new dependency etc.
Basically it's much bigger in scope than originally thought. As such it probably requires some discussion with maintainers before actually proceeding with an implementation. So a simple docs change PR wouldn't cut it 😕

@shellscape
Copy link
Collaborator

Folks, I really don't mean for this to be curt; if the lack of a hyperlink is stopping you from following the issue number, you may want to consider slowing down a tad, use the problem solving skills that got you into the industry to discern how to view the PR or issue number. In terms of blockers, that's the softest one I've ever seen.

We'll update the script so it creates hyperlinks, but do try and take initiative to learn the platform you're using.

@rollup rollup locked and limited conversation to collaborators Sep 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants