-
Notifications
You must be signed in to change notification settings - Fork 3
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
Feature/update workflows #364
Conversation
…ell-animated/simularium-website into feature/update-workflows
…ium-website into feature/update-workflows
jest coverage report 🧪Total coverage
|
gh-pages/index.html
Outdated
</head> | ||
<body> | ||
<a href="https://simularium.github.io/simularium-website/dev">App with viewer at development head</a> | ||
<a href="https://simularium.github.io/simularium-website/stable">App with latest released stable viewer</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would relative urls "dev" and "stable" work here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh yeah they would
@@ -1,11 +1,8 @@ | |||
name: Deploy AWS production | |||
name: Continuos Integration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Continuous
gh-pages/index.html
Outdated
</head> | ||
<body> | ||
<a href="/dev">App with viewer at development head</a> | ||
<a href="/stable">App with latest released stable viewer</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm not 100% sure it works with the leading / ... unless you tested it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is super cool and I can't wait to see it for real. Thanks for doing this.
Problem
Closes #352
Solution
Using github actions upload and download, I was able to build the app twice with two different builds and then upload them both to the gh-pages branch. I tested this on https://github.com/meganrm/react-redux-github-actions/actions/runs/3085116920 because I couldn't get this action to trigger while it's on a branch.
I also used conditionals to remove the redundant workflow files
Type of change
Please delete options that are not relevant.