Skip to content

Fix YAML linting issues in dependency-review workflow#15

Merged
vannu07 merged 2 commits intovannu07-patch-3from
copilot/sub-pr-14
Feb 2, 2026
Merged

Fix YAML linting issues in dependency-review workflow#15
vannu07 merged 2 commits intovannu07-patch-3from
copilot/sub-pr-14

Conversation

Copy link
Contributor

Copilot AI commented Feb 2, 2026

Addresses YAML linting violations and security best practices in the dependency review workflow configuration.

Changes:

  • Quoted on key as "on": for YAML parser compatibility
  • Normalized branches array to ["main"] (removed excess whitespace)
  • Fixed indentation of commented configuration options to align with active keys under with: block
  • Pinned action references to immutable commit SHAs with release tag annotations:
    • actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
    • actions/dependency-review-action@3c4e3dcb1aa7874d2c16be7d79418e9b7efd6261 # v4.8.2

Before:

on:
  pull_request:
    branches: [ "main" ]
    
steps:
  - uses: actions/checkout@v4
  - uses: actions/dependency-review-action@v4
    with:
      comment-summary-in-pr: always
    #   fail-on-severity: moderate

After:

"on":
  pull_request:
    branches: ["main"]
    
steps:
  - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5  # v4.3.1
  - uses: actions/dependency-review-action@3c4e3dcb1aa7874d2c16be7d79418e9b7efd6261  # v4.8.2
    with:
      comment-summary-in-pr: always
      # fail-on-severity: moderate

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@vannu07 vannu07 changed the title [WIP] Add Dependency Review Action workflow Add Dependency Review Action workflow Feb 2, 2026
Co-authored-by: vannu07 <161148074+vannu07@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 2, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/actions/checkout/releases
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/actions/dependency-review-action/releases
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title Add Dependency Review Action workflow Fix YAML linting issues in dependency-review workflow Feb 2, 2026
Copilot AI requested a review from vannu07 February 2, 2026 12:02
@vannu07 vannu07 marked this pull request as ready for review February 2, 2026 12:13
@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

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

  • 🔍 Trigger a full review

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

@vannu07 vannu07 merged commit 386c97d into vannu07-patch-3 Feb 2, 2026
2 checks passed
@vannu07 vannu07 deleted the copilot/sub-pr-14 branch February 2, 2026 12:13
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