-
Notifications
You must be signed in to change notification settings - Fork 201
REVIT-249324/249325/249329: Migrate DynamoRevit to .NET 10 (Revit 2025) #3307
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
Changes from all commits
7765583
399a48c
31c34ec
0b8c619
6b4c7d3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -225,3 +225,5 @@ test/SystemInJson | |
|
|
||
| # Icon resources | ||
| /src/DynamoRevitIcons/*.resources | ||
|
|
||
| logs/* | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| { | ||
| "sdk": { | ||
| "version": "10.0.100", | ||
| "rollForward": "minor" | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -1,12 +1,10 @@ | ||||||||
| <Project> | ||||||||
| <Import Project="$(SolutionDir)Config/user_local.props" /> | ||||||||
| <PropertyGroup> | ||||||||
| <Platforms>NET70;NET80</Platforms> | ||||||||
| <Platforms>NET100</Platforms> | ||||||||
| <PlatformTarget >x64</PlatformTarget> | ||||||||
| <TargetFramework Condition="'$(Platform)' == 'NET80'">net8.0-windows</TargetFramework> | ||||||||
| <TargetFramework Condition="'$(Platform)' == 'NET70'">net7.0-windows</TargetFramework> | ||||||||
| <TargetFramework Condition="'$(Platform)' == 'NET60'">net6.0-windows</TargetFramework> | ||||||||
| <TargetFramework Condition="'$(Platform)' == 'net48'">net48</TargetFramework> | ||||||||
| <TargetFramework Condition="'$(Platform)' == 'NET100'">net10.0-windows</TargetFramework> | ||||||||
| <DCoreLibSubFolder>net10.0</DCoreLibSubFolder> | ||||||||
|
Comment on lines
+6
to
+7
|
||||||||
| <VisualStudioVersion Condition="'$(VisualStudioVersion)' == ''">17.0</VisualStudioVersion> | ||||||||
| <RevitVersionNumber Condition=" '$(RevitVersionNumber)' == '' ">2025</RevitVersionNumber> | ||||||||
| <REVIT_VERSION>Revit_$(RevitVersionNumber)</REVIT_VERSION> | ||||||||
|
|
@@ -20,22 +18,19 @@ | |||||||
|
|
||||||||
| <DYNAMOTESTAPI Condition=" '$(DYNAMOTESTAPI)' == '' ">C:\Program Files\Dynamo 0.7</DYNAMOTESTAPI> | ||||||||
|
|
||||||||
| <DCoreLibSubFolder Condition="'$(Platform)' == 'NET80'">net8.0</DCoreLibSubFolder> | ||||||||
| <DCoreLibSubFolder Condition="'$(Platform)' == 'NET70'">net6.0</DCoreLibSubFolder> | ||||||||
| <DCoreLibSubFolder Condition="'$(Platform)' == 'NET60'">net6.0</DCoreLibSubFolder> | ||||||||
| <REVITAPI Condition=" !Exists('$(REVITAPI)') ">$(SolutionDir)..\lib\Revit $(RevitVersionNumber)\$(DCoreLibSubFolder)</REVITAPI> | ||||||||
|
||||||||
| <REVITAPI Condition=" !Exists('$(REVITAPI)') ">C:\Program Files\Autodesk\Revit Architecture $(RevitVersionNumber)</REVITAPI> | ||||||||
| <REVITAPI Condition=" !Exists('$(REVITAPI)') ">C:\Program Files\Autodesk\Revit $(RevitVersionNumber)</REVITAPI> | ||||||||
| <REVITAPI Condition=" !Exists('$(REVITAPI)') ">C:\Program Files\Autodesk\Revit Preview Release</REVITAPI> | ||||||||
| <REVITAPI Condition=" !Exists('$(REVITAPI)') ">C:\Program Files\Autodesk\Revit 2025</REVITAPI> | ||||||||
|
||||||||
| <REVITAPI Condition=" !Exists('$(REVITAPI)') ">C:\Program Files\Autodesk\Revit 2025</REVITAPI> |
Copilot
AI
Apr 15, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This adds broad suppression of multiple CS30xx warnings (typically CLS compliance-related). If these are intentional, consider documenting why (e.g., with an XML comment near the property), or scoping the suppression to the specific projects that generate the warnings (instead of the shared SDK props) to avoid hiding new warnings introduced later.
| <NoWarn>MSB3539;CS3001;CS3002;CS3003;CS3009;CS3016;CS3027</NoWarn> | |
| <!-- Keep shared warning suppression narrowly scoped; add any CLS-related CS30xx suppressions in the specific project file that requires them. --> | |
| <NoWarn>MSB3539</NoWarn> |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,21 +2,20 @@ | |
| "nuget": { | ||
| "references": { | ||
| "DynamicLanguageRuntime": "1.2.2", | ||
| "DynamoVisualProgramming.Core": "3.3.1.7726", | ||
| "DynamoVisualProgramming.DynamoCoreNodes": "3.3.1.7726", | ||
| "DynamoVisualProgramming.DynamoServices": "3.3.1.7726", | ||
| "DynamoVisualProgramming.Tests": "3.3.1.7726", | ||
| "DynamoVisualProgramming.WpfUILibrary": "3.3.1.7726", | ||
| "DynamoVisualProgramming.ZeroTouchLibrary": "3.3.1.7726", | ||
| "DynamoVisualProgramming.DynamoSamples": "3.3.1.7726", | ||
| "DynamoVisualProgramming.Core": "3.3.2.11986", | ||
| "DynamoVisualProgramming.DynamoCoreNodes": "3.3.2.11986", | ||
| "DynamoVisualProgramming.DynamoServices": "3.3.2.11986", | ||
| "DynamoVisualProgramming.Tests": "3.3.2.11986", | ||
| "DynamoVisualProgramming.WpfUILibrary": "3.3.2.11986", | ||
| "DynamoVisualProgramming.ZeroTouchLibrary": "3.3.2.11986", | ||
| "DynamoVisualProgramming.DynamoSamples": "3.3.2.11986", | ||
|
Comment on lines
+5
to
+11
|
||
| "Greg": "3.0.1.4707", | ||
| "GregRevitAuth": "3.0.8935.26399", | ||
| "pythonnet_py38_win": "2.5.1", | ||
| "Newtonsoft.Json": "13.0.1", | ||
| "NUnit": "3.13.3", | ||
| "Prism": "4.1.0", | ||
| "RestSharp": "112.0.0", | ||
| "System.Resources.Extensions": "5.0.0" | ||
| "RestSharp": "112.0.0" | ||
| } | ||
| } | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR description says "Added global.json with SDK 10.0.0", but the actual pinned SDK is "10.0.100". Please align the PR description (or the file) so the expected toolchain is unambiguous.