Skip to content

ci: Replace reviewdog with sarif2gha. (#39) - #40

Merged
MinoruSekine merged 2 commits into
mainfrom
ci/39/reviewdog_to_sarif2gha
Jul 11, 2026
Merged

ci: Replace reviewdog with sarif2gha. (#39)#40
MinoruSekine merged 2 commits into
mainfrom
ci/39/reviewdog_to_sarif2gha

Conversation

@MinoruSekine

@MinoruSekine MinoruSekine commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Closes #39

Summary by CodeRabbit

  • Chores
    • Improved automated code-quality checks by updating how PowerShell analysis results appear as pull request annotations.
    • Hardened workflow checkout steps by preventing credential persistence during linting and validation jobs.

@MinoruSekine MinoruSekine self-assigned this Jul 11, 2026
@MinoruSekine MinoruSekine added the enhancement New feature or request label Jul 11, 2026
@coderabbitai

coderabbitai Bot commented Jul 11, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: MinoruSekine/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0e045555-02c2-4bf2-82fe-93638f36843a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • ✅ Review completed - (🔄 Check again to review again)
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/39/reviewdog_to_sarif2gha

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/lint.yml:
- Around line 31-39: Update the sarif2gha command in the workflow step to
reference PSScriptAnalyzer.sarif with the exact casing used by the generated
file, replacing the lowercase psscriptanalyzer.sarif argument.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository: MinoruSekine/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e30bd07b-c692-485b-af67-ed7ee17c1e5e

📥 Commits

Reviewing files that changed from the base of the PR and between 0474ca9 and a49f3fc.

📒 Files selected for processing (1)
  • .github/workflows/lint.yml

Comment thread .github/workflows/lint.yml
@MinoruSekine
MinoruSekine force-pushed the ci/39/reviewdog_to_sarif2gha branch from a49f3fc to ae62973 Compare July 11, 2026 01:06
@MinoruSekine
MinoruSekine merged commit 35535dc into main Jul 11, 2026
5 checks passed
@MinoruSekine
MinoruSekine deleted the ci/39/reviewdog_to_sarif2gha branch July 11, 2026 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: Replace reviewdog with sarif2gha.

1 participant