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

Feature/update workflows #364

Merged
merged 15 commits into from
Oct 3, 2022
Merged

Feature/update workflows #364

merged 15 commits into from
Oct 3, 2022

Conversation

meganrm
Copy link
Contributor

@meganrm meganrm commented Sep 19, 2022

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.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

@meganrm meganrm requested a review from a team as a code owner September 19, 2022 20:44
@meganrm meganrm requested review from toloudis, frasercl and ascibisz and removed request for a team September 19, 2022 20:44
@github-actions
Copy link

github-actions bot commented Sep 19, 2022

jest coverage report 🧪

Total coverage

Status Category Percentage Covered / Total
🟡 Statements 73.85% 517/700
🟡 Branches 68.18% 105/154
🔴 Functions 42.61% 75/176
🟡 Lines 73.77% 498/675

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

</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>
Copy link
Contributor

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?

Copy link
Contributor Author

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
Copy link
Contributor

Choose a reason for hiding this comment

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

Continuous

</head>
<body>
<a href="/dev">App with viewer at development head</a>
<a href="/stable">App with latest released stable viewer</a>
Copy link
Contributor

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

Copy link
Contributor

@toloudis toloudis left a 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.

@meganrm meganrm requested a review from blairlyons September 20, 2022 01:58
@meganrm meganrm merged commit f9c11ca into main Oct 3, 2022
@meganrm meganrm deleted the feature/update-workflows branch October 3, 2022 19:51
@meganrm meganrm mentioned this pull request Dec 2, 2022
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.

Change nightly build to use main branch
3 participants