Skip to content

update readme to include instructions for updating the website#77

Closed
yqtian-se wants to merge 3 commits into
acmsigsoft:mainfrom
yqtian-se:main
Closed

update readme to include instructions for updating the website#77
yqtian-se wants to merge 3 commits into
acmsigsoft:mainfrom
yqtian-se:main

Conversation

@yqtian-se
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

Preview deployed at: https://yqtian.com/acmsigsoft.github.io/pr-77

@CaptainEmerson CaptainEmerson requested a review from Copilot August 22, 2025 23:12
@CaptainEmerson CaptainEmerson marked this pull request as ready for review August 22, 2025 23:14
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@github-actions
Copy link
Copy Markdown

Preview deployed at: https://yqtian.com/acmsigsoft.github.io/pr-77

* Update pr-preview.yml

* Isolate PR previews to prevent overwriting production site

The previous pull request preview workflow deployed directly to the
      GitHub Pages environment, completely overwriting the existing site
      content. This meant that generating a PR preview would take down the live
      production site.

This commit modifies the preview workflow to make it non-destructive:

- The workflow now first checks out the `main` branch and builds the full
production site into the `/public` directory.
- It then checks out the pull request's head branch and builds the 
specific PR preview into a subdirectory (e.g., `/public/pr-123`).
- The combined result is then deployed, ensuring that the production site
and the PR preview can coexist.
@github-actions
Copy link
Copy Markdown

Preview deployed at: https://yqtian.com/acmsigsoft.github.io/pr-77

@yqtian-se
Copy link
Copy Markdown
Contributor Author

Something is wrong here. I will close it now and reopen it after I fix it.

@yqtian-se yqtian-se closed this Aug 23, 2025
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.

2 participants