Skip to content

feat: Add validate command - #1114

Merged
bluekeyes merged 2 commits into
palantir:developfrom
AtzeDeVries:add-validate
Dec 2, 2025
Merged

bluekeyes merged 2 commits into
palantir:developfrom
AtzeDeVries:add-validate

Conversation

@AtzeDeVries

Copy link
Copy Markdown
Contributor

This is an implementation of adding validate command to policybot. This will implement #862.

FOr now it is a bit rough but if this is the way we want to go we can make it a big nicer. It copies we way how validate is handled as a request.

Before this PR

Validations of policy required access to a running policybot server. Running a server requires a github applicaton or a mock of a github application.

After this PR

A user can run policy-bot validate -p < path to policy file>. This is useful if you want to run it in CI.

Possible downsides?

It adds a little bit of code to maintain i guess.

This is an implementation of adding validate command to policybot. This
will implement palantir#862.

FOr now it is a bit rough but if this is the way we want to go we can
make it a big nicer. It copies we way how validate is handled as a
request.

Signed-off-by: Atze de Vries <atze.wiebe.de.vries@coop.no>
@changelog-app

changelog-app Bot commented Nov 22, 2025 •

Copy link
Copy Markdown

Generate changelog in changelog/@unreleased

Type (Select exactly one)

  • Feature (Adding new functionality)
  • Improvement (Improving existing functionality)
  • Fix (Fixing an issue with existing functionality)
  • Break (Creating a new major version by breaking public APIs)
  • Deprecation (Removing functionality in a non-breaking way)
  • Migration (Automatically moving data/functionality to a new system)

Description

Add validate subcommand to validate local policy files

Check the box to generate changelog(s)

  • Generate changelog entry

Comment thread cmd/validate.go Resolved
Co-authored-by: Pravin Dahal <mail@pravindahal.com>
@changelog-app

changelog-app Bot commented Dec 2, 2025

Copy link
Copy Markdown

✅ Successfully generated changelog entry!

Need to regenerate?

Simply interact with the changelog bot comment again to regenerate these entries.


📋Changelog Preview

✨ Features

  • Add validate subcommand to validate local policy files (#1114)

@bluekeyes bluekeyes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@bluekeyes
bluekeyes merged commit adb1cef into palantir:develop Dec 2, 2025
9 checks passed
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.

3 participants