Skip to content

Commit

Permalink
chore(build): attach to workspace (#3449)
Browse files Browse the repository at this point in the history
  • Loading branch information
sedghi authored Jun 6, 2023
1 parent 0954fcb commit aaae33c
Showing 1 changed file with 12 additions and 17 deletions.
29 changes: 12 additions & 17 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,7 @@ jobs:
# Update yarn
- run: yarn -v
# Checkout code and ALL Git Tags
- checkout:
post:
- git fetch --all
- checkout
- restore_cache:
name: Restore Yarn and Cypress Package Cache
keys:
Expand Down Expand Up @@ -102,10 +100,7 @@ jobs:
- run: yarn config set workspaces-experimental true

# Checkout code and ALL Git Tags
- checkout:
post:
- git fetch --all

- checkout
- restore_cache:
name: Restore Yarn and Cypress Package Cache
keys:
Expand Down Expand Up @@ -139,9 +134,7 @@ jobs:
<<: *defaults
steps:
# Checkout code and ALL Git Tags
- checkout:
post:
- git fetch --all
- checkout
- restore_cache:
name: Restore Yarn and Cypress Package Cache
keys:
Expand Down Expand Up @@ -247,13 +240,11 @@ jobs:
NPM_PUBLISH:
<<: *defaults
steps:
- attach_workspace:
at: ~/repo
- run: yarn -v
# Checkout code and ALL Git Tags
- checkout:
post:
- git fetch --all
- checkout
- attach_workspace:
at: ~/repo
# Use increasingly general patterns to restore cache
- restore_cache:
name: Restore Yarn and Cypress Package Cache
Expand Down Expand Up @@ -304,6 +295,10 @@ jobs:
name: publish package dist
command: |
node ./publish-package.mjs
- persist_to_workspace:
root: ~/repo
paths:
- .

DOCKER_RELEASE_PUBLISH:
<<: *defaults
Expand Down Expand Up @@ -449,7 +444,7 @@ workflows:
- BUILD
- DOCKER_BETA_PUBLISH:
requires:
- BUILD
- NPM_PUBLISH

# Our release branch deploys to viewer.ohif.org and is more stable and less
# frequently updated after being tested in the staging environment.
Expand All @@ -464,7 +459,7 @@ workflows:
- BUILD
- DOCKER_RELEASE_PUBLISH:
requires:
- BUILD
- NPM_PUBLISH
###
# Unit and E2E tests have already run for PR_CHECKS
# Re-running should not gain us any confidence here
Expand Down

0 comments on commit aaae33c

Please sign in to comment.