Skip to content

(#728) update omnisharp on every update of cake.bakery#729

Merged
gep13 merged 1 commit intocake-build:hotfix/2.1.1from
nils-a:feature/GH-728
Oct 11, 2022
Merged

(#728) update omnisharp on every update of cake.bakery#729
gep13 merged 1 commit intocake-build:hotfix/2.1.1from
nils-a:feature/GH-728

Conversation

@nils-a
Copy link
Copy Markdown
Member

@nils-a nils-a commented Oct 11, 2022

fixes #728

@nils-a nils-a requested review from bjorkstromm and gep13 October 11, 2022 11:29
Copy link
Copy Markdown
Member

@bjorkstromm bjorkstromm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor... How do I test it? Do we publish VSIX on PR build?

@nils-a
Copy link
Copy Markdown
Member Author

nils-a commented Oct 11, 2022

checkout, build.ps1, in VSCode: "Extensions: Install from VSIX".

We should probably look into uploading the VSIX as an artifact, or publishing previews to https://www.vsixgallery.com (though I'm not sure it hosts vscode VSIX)

@gep13 gep13 changed the base branch from develop to hotfix/2.1.1 October 11, 2022 19:40
Copy link
Copy Markdown
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@gep13 gep13 dismissed bjorkstromm’s stale review October 11, 2022 20:10

Suggested changes have been implemented.

Copy link
Copy Markdown
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@gep13 gep13 merged commit 50d3a50 into cake-build:hotfix/2.1.1 Oct 11, 2022
@gep13
Copy link
Copy Markdown
Member

gep13 commented Oct 11, 2022

@nils-a thank you very much for getting this fixed up!

@nils-a nils-a deleted the feature/GH-728 branch October 11, 2022 20:40
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.

omnisharp.json is not updated after manually calling the "Update Intellisense support" command

3 participants