Skip to content

Docs auto update#43

Open
joaopapereira wants to merge 3 commits into
carvel-dev:mainfrom
joaopapereira:docs-auto-update
Open

Docs auto update#43
joaopapereira wants to merge 3 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
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 on lines +209 to +212
echo "${{ secrets.INSTALL_SH_DEPLOY_PRIVATE_KEY }}" > "${TMP_DEPLOY_PRIV_KEY}"
chmod 600 "${TMP_DEPLOY_PRIV_KEY}"
eval $(ssh-agent -s)
ssh-add "${TMP_DEPLOY_PRIV_KEY}"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants