Skip to main content

ignored-import

Summary: Reference ignores import

Category: Imports

Avoid

package policy

import data.authz.roles

allow if {
    some role in input.user.roles
    # data.authz.roles has been imported, but the import is ignored here
    role in data.authz.roles.admin_roles
}

Prefer

package policy

import data.authz.roles

allow if {
    some role in input.user.roles
    # imported data.authz.roles used
    role in roles.admin_roles
}

Rationale​

Imports tend to make long, nested references more readable, and encourages reuse of common logic. Using a full reference (like data.users.permissions) despite having previously imported the reference, or parts of it (like data.users) defeats the purpose of the import, and you're better off referring to the import directly.

Configuration Options​

This linter rule provides the following configuration options:

rules:
  imports:
    ignored-import:
      # one of "error", "warning", "ignore"
      level: error