Skip to content

(optim/ci): don't build a second time for lint or test#623

Merged
agilgur5 merged 1 commit into
jaredpalmer:masterfrom
agilgur5:ci-optim-post-build
Mar 17, 2020
Merged

(optim/ci): don't build a second time for lint or test#623
agilgur5 merged 1 commit into
jaredpalmer:masterfrom
agilgur5:ci-optim-post-build

Conversation

@agilgur5

@agilgur5 agilgur5 commented Mar 17, 2020

Copy link
Copy Markdown
Collaborator
  • yarn install already builds, so we can skip the build step of these
    scripts and run the post-build step of each instead

Split out from #621

- yarn install already builds, so we can skip the build step of these
  scripts and run the post-build step of each instead

@agilgur5 agilgur5 left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 two half-line changes. CI has passed here and in the PR it was split out from (#621)

@agilgur5 agilgur5 merged commit 6679c23 into jaredpalmer:master Mar 17, 2020
paul-vd pushed a commit to EezyQuote/tsdx that referenced this pull request Dec 1, 2020
- yarn install already builds, so we can skip the build step of these
  scripts and run the post-build step of each instead
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant