Skip to content

Conversation

AlanRynne
Copy link
Contributor

Adds XX_OR_GREATER to all converter and connector projects:

  • Rhino
  • Revit
  • Acad/Civil

Left ArcGIS out because it has no other version, so it's intention is not as clear.

Copy link

linear bot commented Aug 12, 2024

Copy link

codecov bot commented Aug 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 8.19%. Comparing base (db60e3e) to head (394a5a1).

Additional details and impacted files
@@          Coverage Diff          @@
##             dev    #136   +/-   ##
=====================================
  Coverage   8.19%   8.19%           
=====================================
  Files        233     233           
  Lines       4391    4391           
  Branches     503     503           
=====================================
  Hits         360     360           
  Misses      4021    4021           
  Partials      10      10           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -3,6 +3,7 @@
<PropertyGroup>
<TargetFramework>net48</TargetFramework>
<PlatformTarget>x64</PlatformTarget>
<DefineConstants>$(DefineConstants);CIVIL3D;CIVIL3D2024;CIVIL3D2024_OR_GREATER</DefineConstants>
Copy link
Member

Choose a reason for hiding this comment

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

Guessing we should have CIVIL3D2023_OR_GREATER here too?
Tho, not sure the status of our CIVIL versions... if we don't have other versions yet, then we should just remember to correct the DefineConstants later

@AlanRynne AlanRynne merged commit 647cbb5 into dev Aug 13, 2024
5 checks passed
@AlanRynne AlanRynne deleted the CXPLA-17-Add-OR_GREATER-flags branch August 13, 2024 07:12
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.

3 participants