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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid validation warning #31557

Closed
jrieken opened this issue Jul 27, 2017 · 2 comments
Closed

Invalid validation warning #31557

jrieken opened this issue Jul 27, 2017 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@jrieken
Copy link
Member

jrieken commented Jul 27, 2017

testing #31448

  • create a workspace and add a folder form a UNC drive
  • open workspace file (e.g. via settings)
  • 馃悰 there is warning on the file://authorty/path-uri which isn't valid

screen shot 2017-07-27 at 14 23 44

@vscodebot vscodebot bot added the workbench label Jul 27, 2017
@jrieken jrieken added the bug Issue identified by VS Code Team member as probable bug label Jul 27, 2017
@sandy081 sandy081 added this to the July 2017 milestone Jul 27, 2017
@sandy081
Copy link
Member

@jrieken At present we only support file paths. So the pattern ^file:// is ok?

@jrieken
Copy link
Member Author

jrieken commented Jul 27, 2017

Yeah, should be ok... A third slash is required but not always after the second. Try ^file:\/\/[^/]*\/, it should match file:///local/file and file://shares/file

sandy081 added a commit that referenced this issue Jul 27, 2017
@roblourens roblourens added the verified Verification succeeded label Aug 3, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants