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

Update Visual Basic Project Template (CLI) for 5.0 Preview 8 #3718

Merged
merged 2 commits into from
Aug 11, 2020

Conversation

KlausLoeffelmann
Copy link
Member

@KlausLoeffelmann KlausLoeffelmann commented Aug 10, 2020

Visual Basic will be having a new WinForms project template which works based on the VB Application Framework for Visual Studio 16.9, while the template for CLI should be in the same style as the CS template for Window Forms.

The new VS template for VB WinForms/Application Framework relies on a code generation tool which is only available in VS, so having this template for the CLI version of VB would not work.

The current VB WinForms CLI (dotnet new winforms --language vb) template however does only invoke the Form it created as a start object. It does not setup other necessary settings, like enabling Visual Styles and setting the correct HighDpiMode, but it should.

This PR corrects this.

Microsoft Reviewers: Open in CodeFlow

@codecov
Copy link

codecov bot commented Aug 10, 2020

Codecov Report

Merging #3718 into release/5.0-preview8 will decrease coverage by 31.63288%.
The diff coverage is n/a.

@@                       Coverage Diff                       @@
##           release/5.0-preview8       #3718          +/-   ##
===============================================================
- Coverage              67.08048%   35.44760%   -31.63288%     
===============================================================
  Files                      1360         904         -456     
  Lines                    504418      251419      -252999     
  Branches                  40855       36679        -4176     
===============================================================
- Hits                     338366       89122      -249244     
+ Misses                   160484      157449        -3035     
+ Partials                   5568        4848         -720     
Flag Coverage Δ
#Debug 35.44760% <ø> (-31.63288%) ⬇️
#production 35.44760% <ø> (+0.00079%) ⬆️
#test ?

Flags with carried forward coverage won't be shown. Click here to find out more.

@codecov
Copy link

codecov bot commented Aug 11, 2020

Codecov Report

Merging #3718 into release/5.0-preview8 will decrease coverage by 31.62015%.
The diff coverage is n/a.

@@                       Coverage Diff                       @@
##           release/5.0-preview8       #3718          +/-   ##
===============================================================
- Coverage              67.08048%   35.46033%   -31.62015%     
===============================================================
  Files                      1360         904         -456     
  Lines                    504418      251419      -252999     
  Branches                  40855       36679        -4176     
===============================================================
- Hits                     338366       89154      -249212     
+ Misses                   160484      157418        -3066     
+ Partials                   5568        4847         -721     
Flag Coverage Δ
#Debug 35.46033% <ø> (-31.62015%) ⬇️
#production 35.46033% <ø> (+0.01352%) ⬆️
#test ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link
Member

@Shyam-Gupta Shyam-Gupta left a comment

Choose a reason for hiding this comment

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

Looks good.

@RussKie RussKie merged commit 0416d4a into release/5.0-preview8 Aug 11, 2020
@RussKie RussKie deleted the dev/klloeffe/VbCliTemplateUpdate5-0Pre8 branch August 11, 2020 01:06
@RussKie RussKie added this to the 5.0 Preview8 milestone Aug 11, 2020
@RussKie RussKie added the Servicing-approved .NET Shiproom approved the PR for merge label Aug 11, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Jan 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Servicing-approved .NET Shiproom approved the PR for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants