Skip to content

Propose various improvement to circleci workflow#589

Merged
xhluca merged 6 commits intomasterfrom
improve-circleci
Mar 31, 2021
Merged

Propose various improvement to circleci workflow#589
xhluca merged 6 commits intomasterfrom
improve-circleci

Conversation

@xhluca
Copy link

@xhluca xhluca commented Mar 30, 2021

  • Set create_app=true when deploying the gallery so we don't have to manually create the app beforehand
  • Make runtime.txt optional during deployment
  • Make the assets directory optional if the app does not contain any custom css or images
  • Remake flake8 from tests, since black already covers most of what we want.

xhlulu added 5 commits March 30, 2021 14:15
By only copying the runtime.txt when it exists, this makes it optional for deployment
Change to a log-warn instead of log-fail
@xhluca xhluca marked this pull request as ready for review March 30, 2021 21:22
@xhluca xhluca requested a review from ycaokris March 30, 2021 21:24
Copy link
Contributor

@ycaokris ycaokris left a comment

Choose a reason for hiding this comment

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

LGTM 💃 let's merge and verify that deployment to gallery works as expected.

@xhluca
Copy link
Author

xhluca commented Mar 31, 2021

@xhluca xhluca merged commit 2f8ed51 into master Mar 31, 2021
@xhluca xhluca deleted the improve-circleci branch March 31, 2021 20:05
xhluca pushed a commit that referenced this pull request Jul 13, 2021
Propose various improvement to circleci workflow

Former-commit-id: 2f8ed51
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.

2 participants