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

fix: regex of workspace ID validator #15539

Merged
merged 4 commits into from
Jan 2, 2023
Merged

Conversation

Siddhant-K-code
Copy link
Member

@Siddhant-K-code Siddhant-K-code commented Jan 1, 2023

Signed-off-by: Siddhant Khare [email protected]

Description

Fixes the regex pattern for workspace ID validator 🔧

On this another example Repo.

Before After
image image

Related Issue(s)

Fixes Internal Conversations[1], [2]

How to test

Release Notes

Fix: edge cases on opening workspaces to VS Code Desktop 

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh

@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-fix-workspaceidvalidator.1 because the annotations in the pull request description changed
(with .werft/ from main)

@roboquat roboquat added size/S and removed size/XS labels Jan 1, 2023
@Siddhant-K-code Siddhant-K-code marked this pull request as ready for review January 1, 2023 07:49
@Siddhant-K-code Siddhant-K-code requested a review from a team as a code owner January 1, 2023 07:49
@Siddhant-K-code Siddhant-K-code marked this pull request as ready for review January 2, 2023 12:34
@roboquat roboquat merged commit a8540be into main Jan 2, 2023
@roboquat roboquat deleted the fix/workspaceIdValidator branch January 2, 2023 12:34
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