Skip to content

feat: Update deployment configuration to publish from root directory …#37

Merged
rosasbehoundja merged 1 commit into
mainfrom
feat/notion
Mar 30, 2026
Merged

feat: Update deployment configuration to publish from root directory …#37
rosasbehoundja merged 1 commit into
mainfrom
feat/notion

Conversation

@rosasbehoundja
Copy link
Copy Markdown
Contributor

…and exclude unnecessary assets

Copilot AI review requested due to automatic review settings March 30, 2026 12:34
@rosasbehoundja rosasbehoundja merged commit e4df16f into main Mar 30, 2026
Copy link
Copy Markdown

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.

Pull request overview

Updates the GitHub Pages deployment workflow to publish the site from the repository root while attempting to omit non-site files from the deployed artifact.

Changes:

  • Switch GitHub Pages publish_dir from ./pages to the repository root (.).
  • Add an exclude_assets list to avoid deploying selected non-site files.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

with:
github_token: ${{ secrets.GITHUB_TOKEN }}
publish_dir: ./pages No newline at end of file
publish_dir: .
Copy link

Copilot AI Mar 30, 2026

Choose a reason for hiding this comment

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

With publish_dir: ., the GitHub Pages deployment will include the entire repository root (e.g., data/, docs/, tests/, backup/, lockfiles, etc.) except for the few patterns in exclude_assets. If the intention is to publish only the website (index.html + pages/), consider either deploying a dedicated output directory containing just the site, or expanding exclude_assets to exclude the large/non-site directories that are now being deployed.

Suggested change
publish_dir: .
publish_dir: ./pages

Copilot uses AI. Check for mistakes.
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