Skip to content

Updated TargetFrameworkVersions to match Cake 3.0.0 versions.#218

Merged
nils-a merged 2 commits intocake-contrib:developfrom
squid-box:GH-217
Nov 14, 2022
Merged

Updated TargetFrameworkVersions to match Cake 3.0.0 versions.#218
nils-a merged 2 commits intocake-contrib:developfrom
squid-box:GH-217

Conversation

@squid-box
Copy link
Copy Markdown
Contributor

As noted in #217, the supported frameworks needs to be updated following the release of Cake 3.0.0.

As best as I can tell, this is what was necessary to change - and local tests at least work with the version produced with my changes. I'm very much open to feedback on this, should I have missed/misunderstood something.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Nov 11, 2022

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 11, 2022

Codecov Report

Merging #218 (5d3c0c0) into develop (10515df) will increase coverage by 0.03%.
The diff coverage is 92.30%.

@@             Coverage Diff             @@
##           develop     #218      +/-   ##
===========================================
+ Coverage    94.27%   94.30%   +0.03%     
===========================================
  Files           14       14              
  Lines          629      650      +21     
  Branches        92       94       +2     
===========================================
+ Hits           593      613      +20     
  Misses          16       16              
- Partials        20       21       +1     
Impacted Files Coverage Δ
src/Tasks/TargetFrameworkVersions.cs 91.08% <92.30%> (+0.48%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nils-a
Copy link
Copy Markdown
Contributor

nils-a commented Nov 11, 2022

Thanks for working on this. 👍

Looks good at first glance, I'll have a closer look when I can get to a computer.

@nils-a
Copy link
Copy Markdown
Contributor

nils-a commented Nov 11, 2022

@squid-box would you also be in a position to update the docs?

https://github.com/cake-contrib/CakeContrib.Guidelines/blob/develop/docs/input/guidelines/TargetFramework.md#L29-L40

(Seems I missed those for the 2.0.0 version, too. 😇)

@squid-box
Copy link
Copy Markdown
Contributor Author

@nils-a: I'm really moving into dangerous territory here, documenting something I'm very vaguely understanding 😄 - but I've made an attempt.

I'm not sure if there also needs to be an update to the module version checks for 3.0.0, they're untouched so far (but I did update their documentation as well, for better or worse).

@nils-a
Copy link
Copy Markdown
Contributor

nils-a commented Nov 12, 2022

I'm really moving into dangerous territory here, documenting something I'm very vaguely understanding

I'll guide you, I promise 😆

I'm not sure if there also needs to be an update to the module version checks...

Yes, I'll add some comments to changes you made to the docs. The implementation should reflect that accordingly.

I'll have access to a decent computer (rather: keyboard and monitor) on Monday if you need more help than I can provide via mobile.

squid-box and others added 2 commits November 14, 2022 08:57
To reflect the required TargetFrameworkVersions for Cake version 3.0.0.
To show Cake version 3.0.0 as recommended version.
@nils-a
Copy link
Copy Markdown
Contributor

nils-a commented Nov 14, 2022

@squid-box Perfect. Thanks for getting this done.

I have fixed up the commits a bit (hope you don't mind) and added one additional change (CCG0009 - Suggested Cake version to use)

@nils-a nils-a enabled auto-merge November 14, 2022 08:02
@nils-a nils-a merged commit 493bf53 into cake-contrib:develop Nov 14, 2022
@nils-a
Copy link
Copy Markdown
Contributor

nils-a commented Nov 14, 2022

@squid-box your changes have been merged, thanks for your contribution 👍

@nils-a nils-a linked an issue Nov 14, 2022 that may be closed by this pull request
cake-contrib-bot pushed a commit that referenced this pull request Nov 14, 2022
Merge pull request #218 from squid-box/GH-217

Updated TargetFrameworkVersions to match Cake 3.0.0 versions.
@squid-box squid-box deleted the GH-217 branch November 14, 2022 08:57
@nils-a
Copy link
Copy Markdown
Contributor

nils-a commented Nov 14, 2022

@all-contributors please add @squid-box for code.

@allcontributors
Copy link
Copy Markdown
Contributor

@nils-a

I've put up a pull request to add @squid-box! 🎉

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.

Supported frameworks since Cake 3.0 are net7.0 and net6.0

3 participants