Skip to content

Allow empty configuration values to be supplied from environment#89032

Merged
copybara-service[bot] merged 1 commit intotensorflow:masterfrom
elfprince13:patch-1
Mar 19, 2025
Merged

Allow empty configuration values to be supplied from environment#89032
copybara-service[bot] merged 1 commit intotensorflow:masterfrom
elfprince13:patch-1

Conversation

@elfprince13
Copy link
Copy Markdown
Contributor

For example it may be desirable for CC_OPT_FLAGS to be empty.

For example it may be desirable for `CC_OPT_FLAGS` to be empty.
@google-ml-butler google-ml-butler bot added the size:XS CL Change Size: Extra Small label Mar 11, 2025
@google-cla
Copy link
Copy Markdown

google-cla bot commented Mar 11, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@keerthanakadiri
Copy link
Copy Markdown
Contributor

Hi @elfprince13, Can you please sign the CLA? Many thanks!

@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label Mar 12, 2025
@github-project-automation github-project-automation bot moved this to Assigned Reviewer in PR Queue Mar 12, 2025
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Mar 12, 2025
@github-project-automation github-project-automation bot moved this from Assigned Reviewer to Approved by Reviewer in PR Queue Mar 12, 2025
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Mar 12, 2025
@elfprince13
Copy link
Copy Markdown
Contributor Author

Hi @elfprince13, Can you please sign the CLA? Many thanks!

Hi, I actually already signed it and the CI bot just didn't re-run afterward =)

@mihaimaruseac
Copy link
Copy Markdown
Contributor

So the bot requires someone (contributor or reviewer) to retrigger a scan. From the check page, there's a link for that, when the CLA check fails.

@elfprince13
Copy link
Copy Markdown
Contributor Author

@mihaimaruseac anything remaining before it can be merged?

@copybara-service copybara-service bot merged commit 258d04e into tensorflow:master Mar 19, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from Approved by Reviewer to Closed/Rejected in PR Queue Mar 19, 2025
@github-project-automation github-project-automation bot moved this from Closed/Rejected to Merged in PR Queue Mar 19, 2025
@google-ml-butler google-ml-butler bot removed awaiting review Pull request awaiting review ready to pull PR ready for merge process labels Mar 19, 2025
@mihaimaruseac
Copy link
Copy Markdown
Contributor

Sorry, I was away while you sent the last message, but the delay here was caused by the need to have this PR imported into the internal systems, more CI to run on the result of the import and a different set of approvers. The process is documented somewhat in CONTRIBUTING.md

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

Labels

size:XS CL Change Size: Extra Small

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

5 participants