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

Move the build to .net 5 #2294

Merged
merged 3 commits into from
Feb 21, 2021
Merged

Move the build to .net 5 #2294

merged 3 commits into from
Feb 21, 2021

Conversation

haacked
Copy link
Contributor

@haacked haacked commented Feb 8, 2021

.NET 5 is fine for building projects that target older versions of .NET. This also should fix the AppVeyor build which fails because the latest coverlet.console only supports .NET 5.

@haacked haacked mentioned this pull request Feb 8, 2021
@codecov
Copy link

codecov bot commented Feb 8, 2021

Codecov Report

Merging #2294 (25ede05) into main (9e2618f) will increase coverage by 0.02%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #2294      +/-   ##
==========================================
+ Coverage   65.88%   65.91%   +0.02%     
==========================================
  Files         553      553              
  Lines       14463    14465       +2     
  Branches      846      846              
==========================================
+ Hits         9529     9534       +5     
+ Misses       4775     4772       -3     
  Partials      159      159              
Impacted Files Coverage Δ
Octokit/Http/Connection.cs 63.72% <25.00%> (+1.01%) ⬆️
Octokit/Http/ApiConnection.cs 60.25% <42.85%> (ø)
...tokit.Reactive/Clients/ObservableReleasesClient.cs 100.00% <100.00%> (ø)
Octokit/Clients/ReleasesClient.cs 100.00% <100.00%> (ø)

@haacked
Copy link
Contributor Author

haacked commented Feb 8, 2021

The failures seem to be the same failures in #2293, so it looks like using .net 5 for the build is fine. This fixes the broken AppVeyor build.

.NET 5 is fine for building projects that target older versions of .NET. This also should fix the AppVeyor build which fails because the latest `coverlet.console` only supports .NET 5.
@shiftkey shiftkey merged commit 375c20e into octokit:main Feb 21, 2021
@shiftkey
Copy link
Member

release_notes: Now using .NET 5 to build and publish

@nickfloyd nickfloyd added Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR and removed category: housekeeping labels Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants