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

Streamline deployment #1181

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Argavyon
Copy link
Contributor

@Argavyon Argavyon commented Aug 21, 2024

This supersedes and is largely based on #1064 by @fredden . It's built atop #1180 .

This pull request eliminates such issues by removing the compiled files from the repository, and instead moves the compilation step to the deployment workflow.

In order to complete this work, the source for GitHub Pages should be updated from "deploy from a branch" to "GitHub Actions". It can be found here: https://github.com/pmotschmann/Evolve/settings/pages

You can find a successful deployment using this workflow at https://github.com/Argavyon/Evolve/actions/runs/10495888905

Here's how the deploy works now (this handles npm run build itself, so no need to build locally nor push to the repo).
Just go to the Actions tab, Deploy Website action, and hit the run workflow button!
deploy

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