Skip to content

chore: [CLI-936] Fix bugs from previous update-docs commit#5975

Merged
dotkas merged 4 commits into
mainfrom
dotkas/CLI-936/fix-broken-update-docs-script
Jun 18, 2025
Merged

chore: [CLI-936] Fix bugs from previous update-docs commit#5975
dotkas merged 4 commits into
mainfrom
dotkas/CLI-936/fix-broken-update-docs-script

Conversation

@dotkas
Copy link
Copy Markdown
Contributor

@dotkas dotkas commented Jun 17, 2025

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Commit messages
    are release-note ready, emphasizing
    what was changed, not how.
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High)
  • Highlights breaking API changes (if applicable)
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: ___)
  • Includes product update to be announced in the next stable release notes

What does this PR do?

Fixes bugs that I didn't catch in the #5955. Among other things, commits must be signed, so we're utilizing the Team CLI Bot's keys to do the commit.

Where should the reviewer start?

Same as #5955

How should this be manually tested?

Same as #5955

What's the product update that needs to be communicated to CLI users?

None, infrastructure change.

@dotkas dotkas changed the title Dotkas/cli 936/fix broken update docs script chore: [CLI-936] Fix bugs from previous update-docs commit Jun 17, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 17, 2025

Warnings
⚠️ There are multiple commits on your branch, please squash them locally before merging!
⚠️

"chore: signing commits, though unsure if it works for the github actions user" is too long. Keep the first line of your commit message under 72 characters.

Generated by 🚫 dangerJS against dc91cbc

@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Jun 17, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

code/snyk check is complete. No issues have been found. (View Details)

@dotkas dotkas force-pushed the dotkas/CLI-936/fix-broken-update-docs-script branch from 720b3d8 to 0b80168 Compare June 17, 2025 18:52
@dotkas dotkas force-pushed the dotkas/CLI-936/fix-broken-update-docs-script branch 7 times, most recently from d273704 to 9cf2051 Compare June 17, 2025 20:13
@dotkas dotkas marked this pull request as ready for review June 17, 2025 20:20
@dotkas dotkas requested a review from a team as a code owner June 17, 2025 20:20
@dotkas dotkas force-pushed the dotkas/CLI-936/fix-broken-update-docs-script branch from 9cf2051 to dc91cbc Compare June 17, 2025 20:27
@dotkas dotkas merged commit 458dd9d into main Jun 18, 2025
8 checks passed
@dotkas dotkas deleted the dotkas/CLI-936/fix-broken-update-docs-script branch June 18, 2025 07:10
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