Skip to content

Commit

Permalink
ci(github actions): update cd-server workflow to detect iam/ changes
Browse files Browse the repository at this point in the history
- use 'iam/**' instead of 'iam/*' to match any file path characters, including '/'
- revert cd-server workflow trigger to push > main.
- remove console log from iam
  • Loading branch information
shavinac committed May 2, 2022
1 parent 462e8fa commit e4e72b4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
7 changes: 3 additions & 4 deletions .github/workflows/cd-server.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,9 @@ name: Deploy IAM Server to Review
# trigger: on push to `main` branch, AND only on changes to `iam/*` files
on:
push:
# branches: [main] // TODO temporarily trigger on push to wip branch
branches: [github-actions]
# paths:
# - "iam/*"
branches: [main]
paths:
- "iam/**"

jobs:
build-and-test:
Expand Down
1 change: 0 additions & 1 deletion iam/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,6 @@ app.post("/api/v0.0.0/verify", (req: Request, res: Response): void => {
return void errorRes(res, "Unable to verify payload");
})
.catch((errorReason) => {
console.error("Error verifying credential: ", errorReason);
return void errorRes(res, "Unable to verify payload");
});
});

0 comments on commit e4e72b4

Please sign in to comment.