Skip to content

Add protoc_args attribute to buf_lint_test and buf_breaking_test#50

Merged
srikrsna-buf merged 1 commit intobufbuild:mainfrom
sushain97:main
Nov 29, 2023
Merged

Add protoc_args attribute to buf_lint_test and buf_breaking_test#50
srikrsna-buf merged 1 commit intobufbuild:mainfrom
sushain97:main

Conversation

@sushain97
Copy link
Copy Markdown
Contributor

Closes #37

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Nov 28, 2023

CLA assistant check
All committers have signed the CLA.

@sushain97
Copy link
Copy Markdown
Contributor Author

The CLA assistant doesn't provide a button for me to actually accept the CLA...

image

@srikrsna-buf
Copy link
Copy Markdown
Member

@sushain97 Can you share the OS and browser (with version info) that you tried this on?

@srikrsna-buf
Copy link
Copy Markdown
Member

@sushain97 The commit was made using a different github account @sushain-stripe and the pr was opened by your personal account. That could be the issue

@sushain97
Copy link
Copy Markdown
Contributor Author

@srikrsna-buf wow, great catch! Thanks. Should be fixed now.

@srikrsna-buf srikrsna-buf merged commit 27e9a19 into bufbuild:main Nov 29, 2023
@srikrsna-buf srikrsna-buf mentioned this pull request May 22, 2024
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.

Set --experimental_allow_proto3_optional

3 participants