Skip to content

Commit 3c79184

Browse files
ardatanardatan
committed
Hmm
1 parent 3483db6 commit 3c79184

File tree

1 file changed

+149
-9
lines changed

1 file changed

+149
-9
lines changed

‎patches/@changesets+assemble-release-plan+5.2.3.patch

+149-9
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,156 @@
11
diff --git a/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.dev.js b/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.dev.js
2-
index 3a37c62..a352a93 100644
2+
index 3a37c62..6f0e2a4 100644
33
--- a/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.dev.js
44
+++ b/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.dev.js
5-
@@ -251,7 +251,7 @@ function shouldBumpMajor({
6-
// we check if it is a peerDependency because if it is, our dependent bump type might need to be major.
7-
return depType === "peerDependencies" && nextRelease.type !== "none" && nextRelease.type !== "patch" && ( // 1. If onlyUpdatePeerDependentsWhenOutOfRange set to true, bump major if the version is leaving the range.
8-
// 2. If onlyUpdatePeerDependentsWhenOutOfRange set to false, bump major regardless whether or not the version is leaving the range.
9-
- !onlyUpdatePeerDependentsWhenOutOfRange || !semver__default['default'].satisfies(incrementVersion(nextRelease, preInfo), versionRange)) && ( // bump major only if the dependent doesn't already has a major release.
10-
+ !onlyUpdatePeerDependentsWhenOutOfRange) && ( // bump major only if the dependent doesn't already has a major release.
11-
!releases.has(dependent) || releases.has(dependent) && releases.get(dependent).type !== "major");
12-
}
5+
@@ -75,7 +75,7 @@ function incrementVersion(release, preInfo) {
6+
// because semver.inc with prereleases is confusing and this seems easier
7+
8+
9+
- version += `-${preInfo.state.tag}.${preVersion}`;
10+
+ version += `-next.${preVersion}`;
11+
}
12+
13+
return version;
14+
@@ -130,38 +130,30 @@ function determineDependents({
15+
depType,
16+
versionRange
17+
} of dependencyVersionRanges) {
18+
- if (nextRelease.type === "none") {
19+
- continue;
20+
- } else if (shouldBumpMajor({
21+
- dependent,
22+
- depType,
23+
- versionRange,
24+
- releases,
25+
- nextRelease,
26+
- preInfo,
27+
- onlyUpdatePeerDependentsWhenOutOfRange: config.___experimentalUnsafeOptions_WILL_CHANGE_IN_PATCH.onlyUpdatePeerDependentsWhenOutOfRange
28+
- })) {
29+
- type = "major";
30+
- } else if ((!releases.has(dependent) || releases.get(dependent).type === "none") && (config.___experimentalUnsafeOptions_WILL_CHANGE_IN_PATCH.updateInternalDependents === "always" || !semver__default['default'].satisfies(incrementVersion(nextRelease, preInfo), versionRange))) {
31+
- switch (depType) {
32+
- case "dependencies":
33+
- case "optionalDependencies":
34+
- case "peerDependencies":
35+
- if (type !== "major" && type !== "minor") {
36+
- type = "patch";
37+
- }
38+
-
39+
- break;
40+
-
41+
- case "devDependencies":
42+
- {
43+
- // We don't need a version bump if the package is only in the devDependencies of the dependent package
44+
- if (type !== "major" && type !== "minor" && type !== "patch") {
45+
- type = "none";
46+
+ if (['patch', 'minor', 'major'].indexOf(nextRelease.type) > ['patch', 'minor', 'major'].indexOf(type)) {
47+
+ type = nextRelease.type
48+
+ } else if ( // TODO validate this - I don't think it's right anymore
49+
+ (!releases.has(dependent) || releases.get(dependent).type === "none") && (config.___experimentalUnsafeOptions_WILL_CHANGE_IN_PATCH.updateInternalDependents === "always" || !semver__default['default'].satisfies(incrementVersion(nextRelease, preInfo), versionRange))) {
50+
+ switch (depType) {
51+
+ case "dependencies":
52+
+ case "optionalDependencies":
53+
+ case "peerDependencies":
54+
+ if (type !== "major" && type !== "minor") {
55+
+ type = "patch";
56+
}
57+
- }
58+
+
59+
+ break;
60+
+
61+
+ case "devDependencies":
62+
+ {
63+
+ // We don't need a version bump if the package is only in the devDependencies of the dependent package
64+
+ if (type !== "major" && type !== "minor" && type !== "patch") {
65+
+ type = "none";
66+
+ }
67+
+ }
68+
+ }
69+
}
70+
- }
71+
+
72+
}
73+
}
74+
75+
@@ -174,22 +166,31 @@ function determineDependents({
76+
type,
77+
pkgJSON: dependentPackage.packageJson
78+
};
79+
- }).filter(dependentItem => !!dependentItem.type).forEach(({
80+
+ }).filter(({
81+
+ type
82+
+ }) => type).forEach( // @ts-ignore - I don't know how to make typescript understand that the filter above guarantees this and I got sick of trying
83+
+ ({
84+
name,
85+
type,
86+
pkgJSON
87+
}) => {
88+
- // At this point, we know if we are making a change
89+
- updated = true;
90+
const existing = releases.get(name); // For things that are being given a major bump, we check if we have already
91+
// added them here. If we have, we update the existing item instead of pushing it on to search.
92+
// It is safe to not add it to pkgsToSearch because it should have already been searched at the
93+
// largest possible bump type.
94+
95+
- if (existing && type === "major" && existing.type !== "major") {
96+
- existing.type = "major";
97+
- pkgsToSearch.push(existing);
98+
+ if (existing) {
99+
+ const index = ["patch", "minor", "major"].indexOf(existing.type)
100+
+ if (type === "major") {
101+
+ existing.type = "major";
102+
+ } else if (type === "minor" && index < 1) {
103+
+ existing.type = "minor";
104+
+ } else if (type === "patch" && index < 0) {
105+
+ existing.type = "patch";
106+
+ }
107+
+ // pkgsToSearch.push(existing);
108+
} else {
109+
+ updated = true;
110+
let newDependent = {
111+
name,
112+
type,
113+
@@ -307,20 +308,15 @@ function getHighestReleaseType(releases) {
114+
let highestReleaseType = "none";
115+
116+
for (let release of releases) {
117+
- switch (release.type) {
118+
- case "major":
119+
+ console.log(release, release.type)
120+
+ if (release.type === "major") {
121+
return "major";
122+
-
123+
- case "minor":
124+
+ } else if (release.type === "minor") {
125+
highestReleaseType = "minor";
126+
- break;
127+
-
128+
- case "patch":
129+
+ } else if (release.type === "patch") {
130+
if (highestReleaseType === "none") {
131+
highestReleaseType = "patch";
132+
}
133+
-
134+
- break;
135+
}
136+
}
137+
138+
diff --git a/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.js b/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.js
139+
index f3beb31..42d1290 100644
140+
--- a/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.js
141+
+++ b/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.js
142+
@@ -1,7 +1,7 @@
143+
'use strict';
13144

145+
-if (process.env.NODE_ENV === "production") {
146+
- module.exports = require("./assemble-release-plan.cjs.prod.js");
147+
-} else {
148+
+// if (process.env.NODE_ENV === "production") {
149+
+// module.exports = require("./assemble-release-plan.cjs.prod.js");
150+
+// } else {
151+
module.exports = require("./assemble-release-plan.cjs.dev.js");
152+
-}
153+
+// }
14154
diff --git a/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.prod.js b/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.prod.js
15155
index 87b4c10..a0b45c2 100644
16156
--- a/node_modules/@changesets/assemble-release-plan/dist/assemble-release-plan.cjs.prod.js

0 commit comments

Comments
 (0)