Skip to content

Github Pages Publish Fixes#250

Merged
stevebio merged 1 commit into
developfrom
fixGhPagesPubError
May 19, 2025
Merged

Github Pages Publish Fixes#250
stevebio merged 1 commit into
developfrom
fixGhPagesPubError

Conversation

@stevebio
Copy link
Copy Markdown

@stevebio stevebio commented May 18, 2025

Set github-pages Gem version to latest: 232.
Delete Gemfile.lock. Should be ignored for Github Pages standard Jekyll pages-build-deployment workflow.
Add Gemfile.lock to docs/.gitignore.
Remove custom jekyll workflow. For NiFi docs, using standard pages-build-deployment workflow.

Set github-pages Gem version to latest: 232.
Delete Gemfile.lock. Should be ignored for Github Pages standard Jekyll
Add Gemfile.lock to docs/.gitignore
Remove custom jekyll workflow action. For NiFi docs, using standard Jekyll publish action.
Copilot AI review requested due to automatic review settings May 18, 2025 17:38
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

This PR updates the GitHub Pages Gem version in the docs Gemfile and aligns the configuration with the standard Jekyll publish action.

  • Update GitHub Pages Gem version from "> 228" to "> 232"
  • Supports removal of custom Jekyll workflow action referenced in the description
Files not reviewed (2)
  • .github/workflows/jekyll-gh-pages.yml: Language not supported
  • docs/.gitignore: Language not supported

@stevebio
Copy link
Copy Markdown
Author

I have configured Github Pages to publish from the fixGhPagesPubError branch for the moment. The test worked, it published fine to nifi docs URL.
Once merged into develop, I will create a PR for develop->master and then switch the GHP target branch back to master branch.

@stevebio stevebio changed the title Github Pages Publish Action Fixes Github Pages Publish Fixes May 18, 2025
Copy link
Copy Markdown

@rjrudin rjrudin left a comment

Choose a reason for hiding this comment

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

Based on https://github.com/marklogic/nifi/actions/runs/15098416667 , it looks like this worked? In which case - just set the GH Pages setting back to "main" for the branch that it publishes from.

@stevebio stevebio merged commit dff4a68 into develop May 19, 2025
4 checks passed
@stevebio stevebio deleted the fixGhPagesPubError branch May 19, 2025 15:20
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.

3 participants