-
Notifications
You must be signed in to change notification settings - Fork 205
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
Disallow backslash (windows path) in cadl config #1261
Milestone
Comments
ghost
added
the
Needs Triage
label
Nov 11, 2022
timotheeguerin
added
design:needed
A design request has been raised that needs a proposal
and removed
Needs Triage
labels
Nov 14, 2022
markcowl
removed
the
design:needed
A design request has been raised that needs a proposal
label
Nov 16, 2022
est: 3 |
markcowl
added
the
breaking-change
A change that might cause specs or code to break
label
Jul 30, 2024
Adding the breaking change label, as this could cause new diagnostics for existing specs |
This was reverted as it applied to the CLI as well this is a problem #4173 |
timotheeguerin
removed
the
breaking-change
A change that might cause specs or code to break
label
Aug 13, 2024
for this to work we need the cli to be able to resolve what emitter optins are ahead of time so it can call |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Feel like there has been cases where this was done and caused issues.
Even though it could maybe work this make it always a bad experience having to escape
\
and not being able to copy outside of windows.Think it would just be easier to force using
/
for paths in config.(Emit a diagnostic)The text was updated successfully, but these errors were encountered: