Skip to content

Include optional module name on file descriptor - #18

Merged
pkwarren merged 1 commit into
mainfrom
pkw/module-name
Oct 2, 2026
Merged

pkwarren merged 1 commit into
mainfrom
pkw/module-name

Conversation

@pkwarren

@pkwarren pkwarren commented Oct 2, 2026

Copy link
Copy Markdown
Member

In some cases, it may be useful to know the BSR module name of a file descriptor in a check plugin. This could be used to enforce package naming constraints matching the module's organization name or name.

Update copyright years and buf CLI version to the latest.

@pkwarren
pkwarren requested a review from emcfarlane October 2, 2026 16:09
@pkwarren
pkwarren requested a review from bufdev as a code owner October 2, 2026 16:09
@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedOct 2, 2026, 4:10 PM

@pkwarren pkwarren changed the title Include optional module ame on file descriptor Include optional module name on file descriptor Oct 2, 2026
In some cases, it may be useful to know the BSR module name of a file
descriptor in a check plugin. This could be used to enforce package
naming constraints matching the module's organization name or name.

Update copyright years and buf CLI version to the latest.
Comment thread .github/workflows/ci.yaml
uses: actions/setup-go@v7
with:
go-version: 1.23.x
go-version: 1.27.x

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Needed >= 1.26.7 to install the latest CLI.

(buf.validate.field).ignore = IGNORE_IF_ZERO_VALUE,
(buf.validate.field).string.uri = true
];
string url = 3 [(buf.validate.field).string.uri = true];

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixes a lint warning w/ the latest CLI.

@pkwarren
pkwarren merged commit 30a1a69 into main Oct 2, 2026
5 checks passed
@pkwarren
pkwarren deleted the pkw/module-name branch October 2, 2026 17:02
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