Skip to content

Conversation

@wooln
Copy link
Contributor

@wooln wooln commented Dec 12, 2024

build: add new strong-named assemblies for dtm client

  • Add new projects for creating strong-named assemblies
        - DtmCommon.StrongName, replace Dapper package with Dapper.StrongName
        - Dtmcli.StrongName
        - Dtmgrpc.StrongName
        - Dtmworkflow.StrongName
  • New assembly is copied from .csproj file, and links .cs and .proto file to compile
  • Update solution file to include new projects
  • Adapted to DevOps

Tips: the following dependents do NOT have strong name

  • Dapper
  • Apps72.Dev.Data.DbMocker
  • MongoDB.Bson
  • MongoDB.Driver
  • MongoDB.Driver.Core

solution like this
new strong name assembly

@codecov
Copy link

codecov bot commented Dec 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.82%. Comparing base (8a4bcb1) to head (ecfe2bd).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #89   +/-   ##
=======================================
  Coverage   68.82%   68.82%           
=======================================
  Files          44       44           
  Lines        1915     1915           
  Branches      182      182           
=======================================
  Hits         1318     1318           
  Misses        560      560           
  Partials       37       37           

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

- Add StringNameKey file and StringNamePublicKey properties for assembly signing
- Add new projects for creating strong-named assemblies
    - Dtmcli.StrongName
    - DtmCommon.StrongName, replace Dapper package with Dapper.StrongName
    - Dtmgrpc.StrongName
    - Dtmworkflow.StrongName
- New assembly is copied from .csproj file, and links .cs and .proto file to compile
- Update solution file to include new projects
@wooln wooln force-pushed the feature/sign-assembly branch from 5d556fe to de9f6ad Compare December 12, 2024 08:10
@wooln wooln marked this pull request as ready for review December 12, 2024 08:16
@wooln wooln changed the title build: add strong name signing to library and unit test projects build: add new strong-named assemblies for dtm client DtmCommon, Dtmcli, Dtmgrpc, Dtmworkflow Dec 12, 2024
@wooln wooln force-pushed the feature/sign-assembly branch from e6570d9 to 7064311 Compare December 16, 2024 02:39
@wooln wooln force-pushed the feature/sign-assembly branch from 7064311 to ecfe2bd Compare December 16, 2024 02:44
@catcherwong catcherwong merged commit 40711b8 into dtm-labs:main Dec 18, 2024
4 of 5 checks passed
@wooln wooln deleted the feature/sign-assembly branch December 18, 2024 01:07
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.

2 participants