Skip to content

Conversation

@RajaVarmaGVSSR
Copy link

@RajaVarmaGVSSR RajaVarmaGVSSR commented Oct 1, 2025

Adds an export-ignore rule for buf.yaml to prevent it from appearing in GitHub release
tarballs and zip files.

Background:

  • Release archives currently include all tracked files, which previously caused
    buf.yaml to appear in the tarball.
  • package.json and package-lock.json are no longer in the repo, so no action needed.
  • Existing release workflow remains unchanged.

Closes #604

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 1, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: lmolkova / name: Liudmila Molkova (16b33f0)

@RajaVarmaGVSSR
Copy link
Author

Please consider adding hacktoberfest-accepted label so this PR can count for Hacktoberfest. Thanks!

@RajaVarmaGVSSR
Copy link
Author

@tigrannajaryan, sorry I am new to this. Are there specific reviewers or groups I need to ask for reviews ?

@tigrannajaryan
Copy link
Member

@open-telemetry/specs-sponsors @open-telemetry/technical-committee please review.

Copy link
Member

@lmolkova lmolkova left a comment

Choose a reason for hiding this comment

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

Is there actually a need to remove this file? Does it create any problems?

It's part of the validation and code-generation and we ship proto files as they are in the archive. We expect consumers of the release to generate code. In a generic case, without this file, consumers might not be able to generate code from the proto we provide.

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.

opentelemetry-proto tar archive contains spurious files

3 participants