next-issue 2.5.4 → 2.5.6
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +6 -2
- package/dist/pipeline.mjs +4 -2
- package/package.json +1 -1
package/README.md
CHANGED
|
@@ -476,8 +476,12 @@ The workflow then gives `next-issue` and `@ravshansbox/pi-next-issue` the same
|
|
|
476
476
|
new version and publishes both packages to npm with a
|
|
477
477
|
[trusted publish](https://docs.npmjs.com/trusted-publishers), so no token is
|
|
478
478
|
stored. Only after the publish does it push the tag and make the GitHub release,
|
|
479
|
-
so a failed publish leaves no half-release behind.
|
|
480
|
-
|
|
479
|
+
so a failed publish leaves no half-release behind. npm makes a new version
|
|
480
|
+
live a few minutes after the publish, and not always in the order it got them,
|
|
481
|
+
so before it publishes, the job waits until npm serves both packages at the
|
|
482
|
+
version of the last tag, for up to 10 minutes. Thus a new version never goes
|
|
483
|
+
live before the one that came before it. A manual run of the workflow checks
|
|
484
|
+
out the last tag and publishes that commit under the tag version, and
|
|
481
485
|
changes nothing else. This repairs a release that reached GitHub but not npm.
|
|
482
486
|
The release job stays on a GitHub runner, because npm takes provenance only
|
|
483
487
|
from a GitHub runner.
|
package/dist/pipeline.mjs
CHANGED
|
@@ -73,13 +73,15 @@ export async function pendingIssues(context) {
|
|
|
73
73
|
if (listed.has(number)) {
|
|
74
74
|
continue;
|
|
75
75
|
}
|
|
76
|
-
const saved = await readState(repo, number).catch(() =>
|
|
76
|
+
const saved = await readState(repo, number).catch(() => 'broken');
|
|
77
77
|
if (saved === undefined) {
|
|
78
78
|
continue;
|
|
79
79
|
}
|
|
80
80
|
try {
|
|
81
81
|
const { open, ...issue } = await ports.viewIssue(repo, number);
|
|
82
|
-
if (
|
|
82
|
+
if (saved === 'broken' ||
|
|
83
|
+
open ||
|
|
84
|
+
(saved.pr !== undefined && (await ports.prMerged(repo, saved.pr)))) {
|
|
83
85
|
issues.push(issue);
|
|
84
86
|
}
|
|
85
87
|
}
|