Skip to content

Commit

Permalink
chore: fix minor workflow issues (#365)
Browse files Browse the repository at this point in the history
  • Loading branch information
xiehan authored Dec 4, 2023
1 parent c37e055 commit a3fc700
Show file tree
Hide file tree
Showing 7 changed files with 22 additions and 16 deletions.
1 change: 1 addition & 0 deletions .github/workflows/auto-approve.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions .projen/deps.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions .projen/tasks.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions .projenrc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ const project = new cdk.JsiiProject({
licensed: false, // we do supply our own license file with a custom header
pullRequestTemplate: false,
jsiiVersion: "~5.2.0",
typescriptVersion: "~5.2.0", // should always be the same major/minor as JSII
peerDeps: ["projen@^0.77.2", "constructs@^10.3.0"],
deps: ["change-case", "fs-extra"],
devDeps: ["@types/fs-extra", "glob"],
Expand Down
2 changes: 2 additions & 0 deletions src/auto-approve.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@ export class AutoApprove {
},
});

(workflow.concurrency as any) = "${{ github.workflow }}-${{ github.ref }}";

const maintainerStatuses = `fromJSON('["OWNER", "MEMBER", "COLLABORATOR"]')`;
workflow.addJobs({
approve: {
Expand Down
8 changes: 6 additions & 2 deletions src/force-release.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,9 @@ export class ForceRelease {
{
name: "Setup Node.js",
uses: "actions/setup-node",
with: { "node-version": "16.14.0" },
with: {
"node-version": project.minNodeVersion,
},
},
{
name: "Install dependencies",
Expand Down Expand Up @@ -96,7 +98,9 @@ export class ForceRelease {
{
name: "Setup Node.js",
uses: "actions/setup-node",
with: { "node-version": "16.14.0" },
with: {
"node-version": project.minNodeVersion,
},
},
{
name: "Setup Go",
Expand Down
21 changes: 9 additions & 12 deletions test/__snapshots__/index.test.ts.snap

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit a3fc700

Please sign in to comment.