XS ◾ ♻️ Updating all reference of .NET 8 to .NET 10 #11766
Conversation
…tent into KM/NET-10-Update
…ules.Content into KM6/NET-10-Update
PR Metrics✔ Thanks for keeping your pull request small.
Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs! |
github-actions
bot
changed the title
|
@KostaMadorsky @sylhuang @lukecookssw |
Co-authored-by: Kaha Mason <kahamason@ssw.com.au>
…SSW.Rules.Content into KM6/NET-10-Update
github-actions
bot
added
Age: 🐣 - Young
categories/software-engineering/rules-to-better-windows-forms.mdx
Outdated
Show resolved
Hide resolved
| **Product Owner**: I don't want to upgrade to .NET 10 this Sprint because I want to wait for .NET 11 | ||
|
|
||
| **Developer**: No, we should upgrade to .NET 8 this Sprint because it will help fix users' performance issues. | ||
| **Developer**: No, we should upgrade to .NET 10 this Sprint because it will help fix users' performance issues. |
There was a problem hiding this comment.
Please use a variable for .NET 10, mdx seem to support it and it works for rules:
There was a problem hiding this comment.
In that case, should we:
- Have a centralized variable of [dotnetTarget] and have all rules reference this variable? like a constants.mdx (not sure if thats posssible, will take a look)
- Have the variable present in each rule individually instead?
There was a problem hiding this comment.
I tried applying this change but it ended up breaking the markdown when viewing it locally.
Might attempt this is a separate PR
github-actions
bot
added
Age: 🐥 - Adolescent
| { | ||
| Id = table.Column<int>(type: "int", nullable: false) | ||
| .Annotation("SqlServer:Identity", "1, 1"), | ||
| Name = table.Column<string>(type: "nvarchar(max)", nullable: true) |
There was a problem hiding this comment.
Did they change the signature in net10?
There was a problem hiding this comment.
I guess the old one still exists, they just added more defined overload, otherwise it would break all the old migrations.
github-actions
bot
added
Age: 🐤 - Mature
|
Howzit @KahaMason, This PR has been here a while. Did you know you should avoid merge debt?
Thanks! |
github-actions
bot
added
Age: 🦖 - Extinct
…tent into KM6/NET-10-Update
Co-authored-by: Kosta Madorsky [SSW] <kosta.madorsky@gmail.com>
We're currently updating all our rules from
.NET 8 Migration->.NET 10 MigrationAs such we need to locate and update all the relevant references.
This PR will not blindly update all references, as some have content that needs to be individually updated e.g. figures, examples and images.
.NET 8references to.NET 10⛔ Blocked by: SSWConsulting/SSW.Rules#2376