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: run Node.js markdown formatter on markdown output #98

Merged
merged 1 commit into from
Oct 12, 2021

Conversation

Trott
Copy link
Member

@Trott Trott commented Oct 10, 2021

To increase the probability of producing markdown that will pass the
Node.js linter, use the Node.js linter to format markdown output.

@Trott Trott force-pushed the formatter branch 2 times, most recently from 555b60e to 1370f4c Compare October 10, 2021 03:36
@targos
Copy link
Member

targos commented Oct 10, 2021

Lockfile has conflicts after I merged your other PRs

@Trott Trott mentioned this pull request Oct 10, 2021
To increase the probability of producing markdown that will pass the
Node.js linter, use the Node.js linter to format markdown output.
@Trott
Copy link
Member Author

Trott commented Oct 10, 2021

Lockfile conflicts resolved.

@Trott
Copy link
Member Author

Trott commented Oct 12, 2021

@targos Can we land this and push out a new release? It would have been great to have this available for the release tomorrow, but at least we can get it in for the next one?

@Trott
Copy link
Member Author

Trott commented Oct 12, 2021

@targos Can we land this and push out a new release? It would have been great to have this available for the release tomorrow, but at least we can get it in for the next one?

Oh, wait, at least in theory, you're not a member of this repository any more than I am. I just checked and the collaborators or Fishrock, rvagg, Myles, and XhmikosR. Could probably stand to update the collaborators. Anyway, I'll go ahead and land this at least.

@Trott Trott merged commit 26afb81 into nodejs:main Oct 12, 2021
@Trott Trott deleted the formatter branch October 12, 2021 02:24
@targos
Copy link
Member

targos commented Oct 12, 2021

Oh, I thought this repo belonged to the Release WG.

@Trott
Copy link
Member Author

Trott commented Oct 12, 2021

Oh, I thought this repo belonged to the Release WG.

It probably should. That was my assumption too!

github-actions bot pushed a commit that referenced this pull request Oct 14, 2021
## [2.7.0](v2.6.0...v2.7.0) (2021-10-14)

### Features

* run Node.js markdown formatter on markdown output ([#98](#98)) ([26afb81](26afb81))

### Bug Fixes

* main instead of master ([c6aac22](c6aac22))
* remove build step from release process ([ec2b9c0](ec2b9c0))

### Trivial Changes

* add auto-release workflow ([36ca9e5](36ca9e5))
* add dependabot config ([5f16f7e](5f16f7e))
* **deps:** bump remark-preset-lint-node from 3.2.0 to 3.3.0 ([ed6ff3b](ed6ff3b))
* **deps:** bump remark-stringify from 10.0.0 to 10.0.1 ([73c2046](73c2046))
* escape opening bracket explicitly ([#97](#97)) ([64c1220](64c1220)), closes [/github.com/nodejs/node/pull/40388#issuecomment-939321694](https://github.com/nodejs//github.com/nodejs/node/pull/40388/issues/issuecomment-939321694)
* update package-lock.json ([#102](#102)) ([24112e0](24112e0))
* update standard (and path-parse) ([#96](#96)) ([66b6eef](66b6eef))
* update test workflow ([#100](#100)) ([eaeb938](eaeb938))
@github-actions
Copy link

🎉 This PR is included in version 2.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@richardlau richardlau mentioned this pull request Dec 2, 2021
renawolford6 added a commit to renawolford6/changelog-maker-javascript that referenced this pull request Nov 9, 2022
## [2.7.0](nodejs/changelog-maker@v2.6.0...v2.7.0) (2021-10-14)

### Features

* run Node.js markdown formatter on markdown output ([#98](nodejs/changelog-maker#98)) ([26afb81](nodejs/changelog-maker@26afb81))

### Bug Fixes

* main instead of master ([c6aac22](nodejs/changelog-maker@c6aac22))
* remove build step from release process ([ec2b9c0](nodejs/changelog-maker@ec2b9c0))

### Trivial Changes

* add auto-release workflow ([36ca9e5](nodejs/changelog-maker@36ca9e5))
* add dependabot config ([5f16f7e](nodejs/changelog-maker@5f16f7e))
* **deps:** bump remark-preset-lint-node from 3.2.0 to 3.3.0 ([ed6ff3b](nodejs/changelog-maker@ed6ff3b))
* **deps:** bump remark-stringify from 10.0.0 to 10.0.1 ([73c2046](nodejs/changelog-maker@73c2046))
* escape opening bracket explicitly ([#97](nodejs/changelog-maker#97)) ([64c1220](nodejs/changelog-maker@64c1220)), closes [/github.com/nodejs/node/pull/40388#issuecomment-939321694](https://github.com/nodejs//github.com/nodejs/node/pull/40388/issues/issuecomment-939321694)
* update package-lock.json ([#102](nodejs/changelog-maker#102)) ([24112e0](nodejs/changelog-maker@24112e0))
* update standard (and path-parse) ([#96](nodejs/changelog-maker#96)) ([66b6eef](nodejs/changelog-maker@66b6eef))
* update test workflow ([#100](nodejs/changelog-maker#100)) ([eaeb938](nodejs/changelog-maker@eaeb938))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants