Skip to content

Docs auto update#43

Merged
joaopapereira merged 6 commits into
carvel-dev:mainfrom
joaopapereira:docs-auto-update
Jun 5, 2026
Merged

Docs auto update#43
joaopapereira merged 6 commits into
carvel-dev:mainfrom
joaopapereira:docs-auto-update

Conversation

@joaopapereira
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: João Pereira <joao.pereira@broadcom.com>
Signed-off-by: João Pereira <joao.pereira@broadcom.com>
prepare-documentation-update:
runs-on: ubuntu-latest
needs: [process-release-information,update-releases-yaml]
if: endsWith(github.event.client_payload.tagName, '.0')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Will this also include prereleases? Do we want to include them?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

what do you mean by prereleases?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I do not think that we do prereleases in our tools, but if we do we can look into it later

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 aims to automate documentation updates as part of the published release workflow, and adjusts the generated install.sh template to verify downloaded binaries using a checksum tool available on the target system.

Changes:

  • Add a new GitHub Actions job to auto-run the website documentation release script and push changes to the website repo on .0 releases.
  • Update the install.sh template to select a checksum utility dynamically and use it for verification.

Reviewed changes

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

File Description
scripts/install_sh/install.sh.txt Updates checksum command selection/usage in the generated install script.
.github/workflows/published_release.yml Adds a documentation auto-update job triggered on .0 releases.

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

Comment thread scripts/install_sh/install.sh.txt Outdated
Comment thread scripts/install_sh/install.sh.txt Outdated
Comment thread .github/workflows/published_release.yml
Comment thread .github/workflows/published_release.yml Outdated
Comment thread .github/workflows/published_release.yml Outdated
Comment thread .github/workflows/published_release.yml Outdated
- Ensure the shasum logic is correct
- Update all the version of the actions being used to the latest

Signed-off-by: Joao Pereira <joaopapereira@gmail.com>
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

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

Comment thread .github/workflows/published_release.yml Outdated
Signed-off-by: Joao Pereira <joaopapereira@gmail.com>
Signed-off-by: Joao Pereira <joaopapereira@gmail.com>
- shell injection protection
- missing popd
- Variable not set

Signed-off-by: Joao Pereira <joaopapereira@gmail.com>
@joaopapereira joaopapereira merged commit daf1170 into carvel-dev:main Jun 5, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this to Closed in Carvel Jun 5, 2026
@joaopapereira joaopapereira deleted the docs-auto-update branch June 5, 2026 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

4 participants