Skip to content

Add a buf.yaml for Buf CLI compatibility - #297

Merged
TristonianJones merged 1 commit into
cel-expr:masterfrom
Alfus:buf
Jul 6, 2023
Merged

Add a buf.yaml for Buf CLI compatibility#297
TristonianJones merged 1 commit into
cel-expr:masterfrom
Alfus:buf

Conversation

@Alfus

@Alfus Alfus commented Jun 30, 2023

Copy link
Copy Markdown
Contributor

To control which files get synced to https://buf.build/cel/spec.

@Alfus

Alfus commented Jun 30, 2023

Copy link
Copy Markdown
Contributor Author

Alternatively, this could be managed by buf.

@TristonianJones

Copy link
Copy Markdown
Collaborator

@Alfus does this help buf.build users more easily integrate with CEL?

@TristonianJones

Copy link
Copy Markdown
Collaborator

/gcbrun

@Alfus

Alfus commented Jul 5, 2023

Copy link
Copy Markdown
Contributor Author

This does allow the buf CLI to run in the cel-spec repo, which allows for local code gen, linting, etc. However, the real reason to include this file, is to control which .proto file descriptors are uploaded to buf.build/cel/spec. Once uploaded, anyone using the buf CLI can access them remotely either by referencing buf.build/cel/spec in a command line argument, or add it as a dependency to a buf.yaml, in a different repo.

@TristonianJones
TristonianJones merged commit 873d4d5 into cel-expr:master Jul 6, 2023
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