Skip to content

Commit ed40123

Browse files
committed
doc: remove "maintenance is supported by" text in BUILDING.md
The "maintenance is supported by" stuff in BUILDING.md is unclear. It seems unnecessary so I propose removing it. I don't understand what it means to, in this context, support maintenance. Does it mean that you simply do the maintenance? Does that mean it really just means "maintain"? Do we really mean that we mantain support, rather than support maintenance? That information does not seem necessary to include. I'm not sure it's meaningful. With (for example) Windows, is it accurate to say that the Node.js core team maintains support for it? Or is it accurate to say that support is maintaned by smaller groups or individuals within the Node.js core team? Both could be considered accurate. So is the difference meaningful? I think the more important elements of determinig tier support are the other listed elements. PR-URL: #30365 Reviewed-By: James M Snell <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent 78dbe74 commit ed40123

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

BUILDING.md

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -69,17 +69,14 @@ There are three support tiers:
6969

7070
* **Tier 1**: These platforms represent the majority of Node.js users. The
7171
Node.js Build Working Group maintains infrastructure for full test coverage.
72-
Maintenance is supported by the Node.js core team. All commits to the
73-
Node.js repository are tested on multiple variants of these platforms. Test
74-
failures on tier 1 platforms will block releases.
72+
All commits to the Node.js repository are tested on multiple variants of these
73+
platforms. Test failures on tier 1 platforms will block releases.
7574
* **Tier 2**: These platforms represent smaller segments of the Node.js user
7675
base. The Node.js Build Working Group maintains infrastructure for full test
77-
coverage. Maintenance is supported by smaller groups or individuals within
78-
the Node.js core team, or the vendor of the platform itself. All commits to
79-
the Node.js repository are tested on multiple variants of these platforms
80-
where practical. Test failures on tier 2 platforms will block releases.
81-
Delays in release of binaries for these platforms are acceptable
82-
where necessary due to infrastructure concerns.
76+
coverage. All commits to the Node.js repository are tested on multiple
77+
variants of these platforms where practical. Test failures on tier 2 platforms
78+
will block releases. Delays in release of binaries for these platforms are
79+
acceptable where necessary due to infrastructure concerns.
8380
* **Experimental**: May not compile or test suite may not pass. The core team
8481
does not create releases for these platforms. Test failures on experimental
8582
platforms do not block releases. Contributions to improve support for these

0 commit comments

Comments
 (0)