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.

Sign up for GitHub

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

Jump to bottom

Modification of omnisharp.json can lead to errors #733

Open
nils-a opened this issue Oct 13, 2022 · 1 comment · May be fixed by #734
Open

Modification of omnisharp.json can lead to errors #733

nils-a opened this issue Oct 13, 2022 · 1 comment · May be fixed by #734

Comments

@nils-a
Copy link
Member

nils-a commented Oct 13, 2022

As discussed on slack:

... Problem is with case-sensitivity. As I've added manually, I used BakeryPath. Cake extension wants bakeryPath.
... Also, the current logic seems to fail if cake section is missing.
nils-a added a commit to nils-a/cake-vscode that referenced this issue Oct 13, 2022
@nils-a
(cake-build#733) use existing properties inside omnisharp.json
fc99053 
instead of enforcing properties "cake" and "bakeryPath"
case sensitive.
@nils-a nils-a linked a pull request Oct 13, 2022 that will close this issue
Open
@Tacodiva
Copy link

Tacodiva commented Feb 14, 2023 •
edited
Loading

Just ran into this, seems like a pretty big issues.
I was trying to use cake for the first time but had an existing omnisharp.json file without the cake property so I was having a pretty hard time getting intellisense working as the extension would just give me errors with cryptic messages. Would be nice to see this merged :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

(#733) use existing properties inside omnisharp.json nils-a/cake-vscode
2 participants
@nils-a @Tacodiva