Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

format: only use ref heads for all rule heads if necessary #5450

Merged
merged 3 commits into from Dec 7, 2022
Merged

format: only use ref heads for all rule heads if necessary #5450

merged 3 commits into from Dec 7, 2022

Commits on Dec 7, 2022

  1. format: only use ref heads for all rule heads if necessary 

    Before, we'd end up formatting

    ps["foo"] = "bar" { true }

as

    ps.foo = "bar" { true }

and older OPA version know how to parse the former, but not
the latter.

Fixes open-policy-agent#5449.

Signed-off-by: Stephan Renatus <stephan.renatus@gmail.com>
    @srenatus
    srenatus committed Dec 7, 2022
    Copy the full SHA
    fba288f View commit details
    Browse the repository at this point in the history

  2. format: pass internal options via struct 

    Signed-off-by: Stephan Renatus <stephan.renatus@gmail.com>
    @srenatus
    srenatus committed Dec 7, 2022
    Copy the full SHA
    a5cf392 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into sr/issue-5449/format-and-backcompat

    @srenatus
    srenatus committed Dec 7, 2022
    Copy the full SHA
    1c88d96 View commit details
    Browse the repository at this point in the history