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

instructions conflict with existing action #2

Open
Tracked by #1
ryanlovett opened this issue Nov 4, 2023 · 2 comments
Open
Tracked by #1

instructions conflict with existing action #2

ryanlovett opened this issue Nov 4, 2023 · 2 comments

Comments

@ryanlovett
Copy link
Contributor

No description provided.

@ryanlovett
Copy link
Contributor Author

ryanlovett commented Nov 4, 2023

Quoted from @paciorek in #1 :

  • Because we say to make a commit before running myst init --gh-pages, MyST tries a deployment that fails based on deploy.yml, so the user will get an email indicating that failure (e.g., here's a failure from my test).
  • When one runs myst init --gh-pages it tells the user they need a new name for the deployment yml file, and then one would have two GH actions.
  • I wonder if we should provide the details of what the user needs to do after myst init --gh-pages.
  • MyST tells the user to "Enable GH pages" but it's not clear what to do to do that. I don't think the user needs to do anything specific for this.
  • MyST says to use GHA as the source. We may want to warn the user that it's ok that this is in Beta.

@ryanlovett ryanlovett changed the title I think we need to move out of the workflows directory in this template repo. If we want to keep it in the repo we could move it aside somewhere else. As it stands, having already exists makes the instructions confusing instructions conflict with existing action Nov 4, 2023
@paciorek
Copy link
Contributor

paciorek commented Nov 6, 2023

Yeah, I thought we could remove the myst init --gh-pages step and tell the user at the step before making any commits to go to https://github.com/example/stat555/settings/pages and under Build and deployment select "GitHub Actions" as the source, noting it is listed as being in beta.

That sort-of works, but the initial commit when the repo is forked on the GH website does trigger a failed action and an email to the user.

We could add a warning to the user that this will happen (and would happen whenever they make a commit if they haven't followed step X (where step X is the one where they set the source for GH pages).

Part of the awkwardness here is that a user might not want the website to be public initially. So we might want to tell the user how to avoid this. We could tell them to move deploy.yml aside until they want things public. Then they would also not get the messages about failed runs.

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

No branches or pull requests

2 participants