Skip to content

Conversation

@developer-guy
Copy link
Member

Fixes:

Related:

Pre-review Checklist

For new package PRs only

  • This PR is marked as fixing a pre-existing package request bug
    • Alternatively, the PR is marked as related to a pre-existing package request bug, such as a dependency
  • REQUIRED - The package is available under an OSI-approved or FSF-approved license
  • REQUIRED - The version of the package is still receiving security updates

For security-related PRs

  • The security fix is recorded in the advisories repo

For version bump PRs

  • The epoch field is reset to 0
  • Patch source: patch source here

@developer-guy developer-guy requested a review from a team as a code owner July 24, 2023 19:10
@developer-guy developer-guy requested review from deitch and kaniini July 24, 2023 19:10
Signed-off-by: Batuhan Apaydin <[email protected]>
Co-authored-by: Furkan Turkal <[email protected]>
Signed-off-by: Batuhan Apaydin <[email protected]>
@kaniini
Copy link
Contributor

kaniini commented Jul 25, 2023

Why are we packaging this? We should not use it for our own internal builds using Bazel, instead we should use our bazel packages for that.

@kaniini kaniini marked this pull request as draft July 25, 2023 00:39
@kaniini
Copy link
Contributor

kaniini commented Jul 25, 2023

Converting to draft until more information is provided as to why this package is needed, as it is most likely not actually needed.

@github-actions
Copy link
Contributor

This Pull Request is stale because it has been open for 90 days with
no activity. It will automatically close after 30 more days of
inactivity. Keep fresh with the 'lifecycle/frozen' label.

@rawlingsj
Copy link
Member

Looks like this is not needed and stale. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants