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

WX-1420 Fix GCP Batch label regex restriction #7353

Closed

Conversation

bchen4
Copy link

@bchen4 bchen4 commented Jan 5, 2024

Issue: #7351

@bchen4 bchen4 marked this pull request as ready for review January 5, 2024 16:28
@bchen4 bchen4 requested a review from a team as a code owner January 5, 2024 16:28
@aednichols aednichols changed the title Fix GCP Batch label regex restriction WX-1420 Fix GCP Batch label regex restriction Jan 5, 2024
@aednichols
Copy link
Collaborator

Hi @bchen4, thanks for the PR.

I created a clone inside the repo to run unit tests: #7353

The tests didn't compile at first which I fixed, but now they are failing and I don't actually know which is wrong, the assertion or the code under test.

I added you as a writer to this repo so you should be able to push to my aen_wx_1420 branch that my PR uses.

@aednichols
Copy link
Collaborator

Merged as #7355

@aednichols aednichols closed this Jan 11, 2024
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.

None yet

5 participants