Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update upb to the latest in 21.x branch #16343

Closed
wants to merge 1 commit into from

Conversation

limdor
Copy link
Contributor

@limdor limdor commented Sep 27, 2022

Some of the commits needed from this branch are:

  • Do not have globs that do not glob anything
  • Do not use platfroms from bazel_tools

This is needed in order to flip the disallowance of empty globs

@limdor limdor mentioned this pull request Sep 27, 2022
56 tasks
@fmeum
Copy link
Collaborator

fmeum commented Sep 27, 2022

There is a special issue template for mirror requests. @sgowroji

@limdor
Copy link
Contributor Author

limdor commented Sep 27, 2022

There is a special issue template for mirror requests. @sgowroji

Oh sorry for that, it seems that I was using outdated information bazelbuild/rules_python#400 (comment)
Could some of you point me to the template? Thanks

@limdor
Copy link
Contributor Author

limdor commented Sep 27, 2022

Ok found, I will fill it https://groups.google.com/g/bazel-discuss/c/nMMztgTGdQA

@sgowroji
Copy link
Member

Hello @limdor, Updated here #16344 (comment)

Some of the commits needed from this branch are:
- Do not have globs that do not glob anything
- Do not use platfroms from bazel_tools

This is needed in order to flip the disallowance of empty globs
@limdor limdor marked this pull request as ready for review September 28, 2022 11:06
@sgowroji sgowroji added team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website awaiting-review PR is awaiting review from an assigned reviewer labels Sep 28, 2022
@meteorcloudy meteorcloudy self-requested a review October 11, 2022 12:52
Copy link
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

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

Sorry for the delay here!

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Oct 18, 2022
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants