Skip to content

Commit 4e3b7c0

Browse files
committed
doc: revise "Breaking Changes to Internal Elements"
Revise the "Breaking Changes to Internal Elements" section of COLLABORATOR_GUIDE.md. Simplify wording, shorten sentences, and remove redundant material.
1 parent 0c1a388 commit 4e3b7c0

File tree

1 file changed

+6
-14
lines changed

1 file changed

+6
-14
lines changed

COLLABORATOR_GUIDE.md

Lines changed: 6 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -269,20 +269,12 @@ For more information, see [Deprecations](#deprecations).
269269

270270
#### Breaking Changes to Internal Elements
271271

272-
Breaking changes to internal elements are permitted in semver-patch or
273-
semver-minor commits but Collaborators should take significant care when
274-
making and reviewing such changes. Before landing such commits, an effort
275-
must be made to determine the potential impact of the change in the ecosystem
276-
by analyzing current use and by validating such changes through ecosystem
277-
testing using the [Canary in the Goldmine](https://github.com/nodejs/citgm)
278-
tool. If a change cannot be made without ecosystem breakage, then TSC review is
279-
required before landing the change as anything less than semver-major.
280-
281-
If a determination is made that a particular internal API (for instance, an
282-
underscore `_` prefixed property) is sufficiently relied upon by the ecosystem
283-
such that any changes may break user code, then serious consideration should be
284-
given to providing an alternative Public API for that functionality before any
285-
breaking changes are made.
272+
Breaking changes to internal elements may be semver-patch or semver-minor.
273+
Collaborators should take significant care when making and reviewing such
274+
changes. An effort must be made to determine the potential impact of the change
275+
in the ecosystem. Use [Canary in the Goldmine](https://github.com/nodejs/citgm)
276+
to test such changes. If a change will cause ecosystem breakage, then it is
277+
semver-major. Consider providing a Public API in such cases.
286278

287279
#### When Breaking Changes Actually Break Things
288280

0 commit comments

Comments
 (0)