Skip to content

Support allow_protocols option for catalog_protocol_usage validation#43

Closed
parksb wants to merge 6 commits intomainfrom
allow-protocols
Closed

Support allow_protocols option for catalog_protocol_usage validation#43
parksb wants to merge 6 commits intomainfrom
allow-protocols

Conversation

@parksb
Copy link
Copy Markdown
Member

@parksb parksb commented Mar 31, 2026

When catalog_protocol_usage is set to strict, any dependency not using the catalog: protocol is flagged as an error. This is a problem for dependencies using protocols like portal:, file:, or link: that serve a different purpose and can't be replaced by catalog:.

This PR adds allow_protocols option that lets users specify protocols from the catalog_protocol_usage validation.

validation:
  - workspaces: ["*"]
    rules:
      catalog_protocol_usage:
        level: strict
        allow_protocols:
          - portal:
          - file:

allow_protocols is only valid with strict or warn, and using it with optional or restrict is rejected as invalid configuration.

@parksb parksb self-assigned this Mar 31, 2026
@parksb parksb closed this Apr 6, 2026
@parksb parksb deleted the allow-protocols branch April 6, 2026 04:16
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.

1 participant