Skip to content

Conversation

@Piskoo
Copy link
Collaborator

@Piskoo Piskoo commented Oct 10, 2025

This PR adds crafting schema v2 that follows structure established by other existing compliance resources.

apiVersion:
kind:
metadata:
  ...
spec:
  ...

chainloop.dev/v1 is used as the only viable api version

Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
@Piskoo Piskoo marked this pull request as ready for review October 10, 2025 07:58
Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
Copy link
Member

@migmartri migmartri left a comment

Choose a reason for hiding this comment

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

LGTM!

Is there any way to mark the other message as deprecated?

// that defines the schema of the workflowRun
message CraftingSchemaV2 {
string api_version = 1 [(buf.validate.field).string.const = "chainloop.dev/v1"];
string kind = 2 [(buf.validate.field).string.const = "Contract"];
Copy link
Member

Choose a reason for hiding this comment

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

nice

@migmartri migmartri merged commit 1718462 into chainloop-dev:main Oct 13, 2025
13 checks passed
@Piskoo
Copy link
Collaborator Author

Piskoo commented Oct 14, 2025

LGTM!

Is there any way to mark the other message as deprecated?

I'll do that in another PR

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