Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Mar 17, 2020

Conversation

agilgur5
Copy link
Collaborator

@agilgur5 agilgur5 commented Mar 17, 2020

  • 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
Copy link
Collaborator Author

@agilgur5 agilgur5 left a comment

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