Skip to content

chore: bump default sinistral cli version to v0.5.39 - #13

Merged
tvansteenburgh merged 1 commit into
mainfrom
ajk/chore/bump-default-cli-version
Oct 7, 2026
Merged

tvansteenburgh merged 1 commit into
mainfrom
ajk/chore/bump-default-cli-version

Conversation

@ajkerrigan

@ajkerrigan ajkerrigan commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

ENG-8511

what

Bumps the sinistral_cli_version default value to the latest sinistral-cli release.

why

To pull in fixes from v0.5.39.

testing

This test ran successfully with these changes, and used the latest sinistral-cli.

image

docs

N/A

@ajkerrigan
ajkerrigan requested a review from a team as a code owner October 7, 2026 19:24

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.
Architecture diagram
sequenceDiagram
    participant Action as GitHub Action
    participant Inputs as Action Inputs
    participant CLI as sinistral-cli
    participant Repo as GitHub Repo

    Note over Action,Repo: sinistral-cli version resolution

    Action->>Inputs: Read sinistral_cli_version input
    Inputs-->>Action: default v0.5.39 (4f2cf75)
    Action->>Action: Determine git ref from input
    alt Custom version provided
        Inputs-->>Action: user-specified ref
    else Default version used
        Inputs-->>Action: v0.5.39 (4f2cf75)
    end
    Action->>Repo: Fetch sinistral-cli at computed ref
    Repo-->>Action: sinistral-cli v0.5.39
    Action->>CLI: Execute scan with resolved version
    CLI-->>Action: Scan results
    Action-->>Action: Post results as PR comment (if enabled)
Loading

Re-trigger cubic

@tvansteenburgh tvansteenburgh 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.

Thanks @ajkerrigan

@tvansteenburgh
tvansteenburgh merged commit 594ed6e into main Oct 7, 2026
3 checks passed
@tvansteenburgh
tvansteenburgh deleted the ajk/chore/bump-default-cli-version branch October 7, 2026 19:54
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