Skip to content

Conversation

@mhdawson
Copy link
Member

@mhdawson mhdawson commented Apr 8, 2021

Signed-off-by: Michael Dawson [email protected]

@DeeDeeG
Copy link

DeeDeeG commented Apr 16, 2021

I think there is an indentation/nesting syntax issue. A couple of extra spaces before the *. Some PRs' main bullet points are nested as if they are discussion bullet points -- interspersed with discussion of other (to my eyes unrelated) PRs.

Details

Under this bullet point:

* doc: mark callback-based fs API as legacy [#37948](https://github.com/nodejs/node/pull/37948)

There are two other PRs indented as if nested under that topic:

* http: significant performance regression on master [#37937](https://github.com/nodejs/node/issues/37937)
* lib: refactor Timeout, ImmediateList, TimersList to class [#37408](https://github.com/nodejs/node/pull/37408)

As someone who did not see the meeting, it is a bit hard to parse out the relationship of said bullet points., and which discussion bullet points go with which PR/topic.

(P.S. thanks for providing minutes and the transparency!)

@Trott
Copy link
Member

Trott commented Apr 17, 2021

I think there is an indentation/nesting syntax issue.

Thanks. I think I fixed it.

@mhdawson mhdawson merged commit a0b9b6a into nodejs:main Apr 20, 2021
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.

7 participants