Skip to content

ci: add GitHub Pages deployment workflow and improve documentation#51

Merged
hrntknr merged 4 commits intomainfrom
docs-workflow-and-improvements
Aug 21, 2025
Merged

ci: add GitHub Pages deployment workflow and improve documentation#51
hrntknr merged 4 commits intomainfrom
docs-workflow-and-improvements

Conversation

@hrntknr
Copy link
Copy Markdown
Member

@hrntknr hrntknr commented Aug 21, 2025

Summary

This PR adds a complete GitHub Pages deployment workflow and improves documentation infrastructure for the project.

Type of Change

  • ci: Changes to our CI configuration files and scripts
  • docs: Documentation only changes

Related Issues

Changes Made

  • GitHub Pages Deployment: Added complete workflow with build and deploy jobs using GitHub Actions
  • Documentation Site: Set up Docusaurus-based documentation with proper structure and configuration
  • CI Enhancements: Enhanced check workflow with documentation linting validation
  • README Improvements: Cleaned up formatting and improved OAuth provider setup documentation
  • Asset Updates: Removed placeholder text from SVG logo

The documentation site will be automatically deployed to GitHub Pages when changes are pushed to main branch.

- Add complete GitHub Pages deployment workflow with build and deploy jobs
- Enhance check workflow with docs linting validation
- Clean up README formatting and improve provider setup documentation
- Remove placeholder text from SVG logo
Copilot AI review requested due to automatic review settings August 21, 2025 12:05
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.

Pull Request Overview

This PR adds a complete GitHub Pages deployment workflow and improves documentation infrastructure for the project by setting up a Docusaurus-based documentation site with automated deployment.

  • Complete GitHub Pages deployment workflow with build and deploy jobs using GitHub Actions
  • Docusaurus-based documentation site with proper structure, configuration, and TypeScript support
  • Enhanced CI workflow with documentation linting validation

Reviewed Changes

Copilot reviewed 18 out of 22 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/docs.yaml New GitHub Actions workflow for building and deploying documentation to GitHub Pages
.github/workflows/check.yaml Enhanced existing CI workflow with documentation linting job
docs/ directory Complete Docusaurus documentation site setup with TypeScript configuration, React components, CSS styling, and markdown content
README.md Minor formatting improvements for OAuth provider setup documentation
.vscode/settings.json VS Code configuration for consistent formatting across file types

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/docs.yaml Outdated
with:
path: docs/build

deploy:
Copy link

Copilot AI Aug 21, 2025

Choose a reason for hiding this comment

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

The deploy job should only run on push to main branch, not on pull requests. Add a condition 'if: github.event_name == 'push' && github.ref == 'refs/heads/main'' to prevent deployment attempts on PR builds.

Suggested change
deploy:
deploy:
if: github.event_name == 'push' && github.ref == 'refs/heads/main'

Copilot uses AI. Check for mistakes.
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Merged the build and deploy jobs into a single job to simplify the workflow
and reduce execution time. The Pages environment configuration is now
properly set on the single job.
Restore the correct workflow name "Deploy Docs to GitHub Pages" and
add necessary permissions for GitHub Pages deployment.
@hrntknr hrntknr merged commit a10a180 into main Aug 21, 2025
7 checks passed
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