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

feat: dropping support for node 10, expanding coverage to node 14/16 #325

Merged
merged 10 commits into from
Jun 21, 2021

Conversation

erunion
Copy link
Member

@erunion erunion commented Jun 9, 2021

Fixes RM-1218
Fixes RM-1219

🧰 Changes

🚨 This'll be a breaking change and will require a bump to rdme@4!

  • Drops support for Node 10
  • Expands our engines, and CI testing, coverage to Node 14 and Node 16.
  • Starts using NPM 7 in CI, and updates the lockfile to the same.
  • Upgrades a handful of dev dependencies to their latest release: eslint, prettier, nock, @readme/eslint-config, jest

🧬 QA & Testing

  • Do all unit tests pass?

@erunion erunion added the enhancement New feature or request label Jun 9, 2021
@erunion erunion force-pushed the feat/drop-node10 branch from 0adf995 to 82e35ac Compare June 18, 2021 23:46
Copy link
Member

@kanadgupta kanadgupta left a comment

Choose a reason for hiding this comment

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

One small q below otherwise LGTM!

package.json Outdated Show resolved Hide resolved
@erunion erunion merged commit 052ed4f into main Jun 21, 2021
@erunion erunion deleted the feat/drop-node10 branch June 21, 2021 20:28
ollyfg pushed a commit to ollyfg/rdme that referenced this pull request Jun 22, 2021
…eadmeio#325)

* feat: dropping support for node 10

* test: fixing a broken test on node 14

* ci: updating the ci workflow to use node 12+

* ci: upgrading the codeql workflow to the latest way it configs

* chore(deps-dev): upgrading jest to v27

* chore(deps-dev): upgrading code styling dev deps

* chore(deps-dev): upgrading nock to the latest version

* chore: removing a swagger file that shouldnt be in the root dir?

* ci: updating codeql to run on `main` not `master`

* fix: removing the `main` declaration because it doesnt exist
@linear
Copy link

linear bot commented Jun 23, 2021

RM-1218 Drop support for Node 10

Drop support for Node 10, ensure that tests run across all LTE releases of Node: 12, 14, 16.

https://nodejs.org/en/about/releases/

@linear
Copy link

linear bot commented Jun 23, 2021

RM-1219 Upgrade the lockfile and CI to use NPM 7

We use NPM 7 mostly everywhere now, so we should update the dev workflow of rdme to be the same. You can use copy the work I did in oas for the work for this:

This'll require us to have to bump to a breaking change with Semver, but I can handle that when we're ready to publish a new release.

rahulhegdee pushed a commit that referenced this pull request Jul 9, 2021
* Add --recursive option to doc sync command

This option causes a simple recursive search of the supplied folder.

* Whoops! Typo

* docs: adding a pull request template

* feat: dropping support for node 10, expanding coverage to node 14/16 (#325)

* feat: dropping support for node 10

* test: fixing a broken test on node 14

* ci: updating the ci workflow to use node 12+

* ci: upgrading the codeql workflow to the latest way it configs

* chore(deps-dev): upgrading jest to v27

* chore(deps-dev): upgrading code styling dev deps

* chore(deps-dev): upgrading nock to the latest version

* chore: removing a swagger file that shouldnt be in the root dir?

* ci: updating codeql to run on `main` not `master`

* fix: removing the `main` declaration because it doesnt exist

* chore: directory spring cleaning (#327)

* feat: dropping support for node 10

* test: fixing a broken test on node 14

* ci: updating the ci workflow to use node 12+

* ci: upgrading the codeql workflow to the latest way it configs

* chore(deps-dev): upgrading jest to v27

* chore(deps-dev): upgrading code styling dev deps

* chore(deps-dev): upgrading nock to the latest version

* chore: removing a swagger file that shouldnt be in the root dir?

* ci: updating codeql to run on `main` not `master`

* chore: updating the license to be 2021

* chore: renaming the test directory to `__tests__`

* chore: various dotfile cleanup

* chore: moving the codebase into a `src/` directory structure

* Bugfix: Get rid of re-declaration

Co-authored-by: Jon Ursenbach <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants