Skip to content

ci(docs): fix docs publishing - #3195

Merged
AlexanderLanin merged 1 commit into
mainfrom
docs-fix
Aug 19, 2026
Merged

AlexanderLanin merged 1 commit into
mainfrom
docs-fix

Conversation

@AlexanderLanin

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Separates documentation building from privileged publishing to support secure previews and Pages deployment.

Changes:

  • Uses read-only permissions for documentation builds.
  • Adds a workflow-run-based publishing pipeline.
  • Updates build triggers and enables manual runs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/docs.yml Configures read-only documentation builds and updated triggers.
.github/workflows/docs-publish.yml Publishes successful build artifacts with scoped write permissions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/docs.yml
Comment thread .github/workflows/docs.yml
@MaximilianSoerenPollak
MaximilianSoerenPollak dismissed their stale review August 19, 2026 12:52

Some quetsions arrose.

@MaximilianSoerenPollak MaximilianSoerenPollak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Questions answered in a call.

@AlexanderLanin
AlexanderLanin merged commit 3257497 into main Aug 19, 2026
8 checks passed
@AlexanderLanin
AlexanderLanin deleted the docs-fix branch August 19, 2026 13:06
@github-project-automation github-project-automation Bot moved this from Backlog to Done in S-CORE Roadmap Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants