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

Update package namespace from rbmViz to gsmViz and add group attributes to all modules. #430

Open
wants to merge 15 commits into
base: dev
Choose a base branch
from

Conversation

samussiah
Copy link
Collaborator

@samussiah samussiah commented Feb 26, 2025

@samussiah samussiah requested a review from jonthegeek February 27, 2025 17:21
uses: JamesIves/github-pages-deploy-action@v4
with:
folder: examples
token: ${{ secrets.GH_PAGES_DEPLOY }}
Copy link
Contributor

Choose a reason for hiding this comment

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

To rough out the "multi deploy" idea:

  • Set a target folder based on trigger type, like I do in the shinyapps deploy workflow.
  • Use that to set the target-folder: variable for this with. Use PR-level folders for PRs, and then probably a dev (in addition to the root for main).
  • We should also have an action to delete those PR folders when the PRs are closed, like I do with the shinyapps terminate workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants