[ 
https://issues.apache.org/jira/browse/AVRO-3469?focusedWorklogId=747256&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-747256
 ]

ASF GitHub Bot logged work on AVRO-3469:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 24/Mar/22 14:31
            Start Date: 24/Mar/22 14:31
    Worklog Time Spent: 10m 
      Work Description: zcsizmadia commented on a change in pull request #1621:
URL: https://github.com/apache/avro/pull/1621#discussion_r834374598



##########
File path: .github/workflows/test-lang-csharp.yml
##########
@@ -63,6 +63,19 @@ jobs:
       - name: Test
         run: ./build.sh test
 
+      # Build and test against .NET 7
+      # .NET 7 is not released yet, however this is a good way to test if the 
project is ready for the release
+      # Once .NET 7 is officially released, this can be removed and 7.0.x can 
be used instead above
+      - name: Install .NET SDK 7.0 (pre-release)
+        uses: actions/setup-dotnet@v1
+        with:
+          include-prerelease: true
+          dotnet-version: |

Review comment:
       In c# we generate a nupkg which has all the supported  .NET SDK version 
compiled. Currently netcoreapp3.1, net5.0, net6.0, which are the MS officially 
supported versions. This is defined in `common.props` by the 
`DefaultExeTargetFrameworks` prop. So the matric approach does not work, 
because all the supported SDKs must be installed in the building/testing image 
and not just one specific version. If the nupkg would be only netstandard we 
could get away with one specific version, however avrogen tool must be compiled 
for all available versions. When that tool is installed, the matching framework 
SDK version is selected from the nupkg.
   E.g. This is the reason I am eager for the 1.11.1 version, because until 
now, I cannot use avrogen in purely/only .NET 6 SDK environment.
   
   So the build step is the same, however when the normal build is done, I 
install .NET SDK and build/test only for net7.0 to check if all is good.
   
   Just fyi, once NEt 7 is released, `DefaultExeTargetFrameworks` will be 
`netcoreapp3.1;net6.0;net7.0` which will reflect the EOL status of the dotnet 
SDKs (https://dotnet.microsoft.com/en-us/download/dotnet)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

            Worklog Id:     (was: 747256)
    Remaining Estimate: 23h 20m  (was: 23.5h)
            Time Spent: 40m  (was: 0.5h)

> Build and test using .NET SDK 7.0 in guthub action
> --------------------------------------------------
>
>                 Key: AVRO-3469
>                 URL: https://issues.apache.org/jira/browse/AVRO-3469
>             Project: Apache Avro
>          Issue Type: Improvement
>            Reporter: Zoltan Csizmadia
>            Priority: Major
>              Labels: pull-request-available
>   Original Estimate: 24h
>          Time Spent: 40m
>  Remaining Estimate: 23h 20m
>
> .NET 7.0 previews are released. Lets put the building and testin on the radar 
> to make sure that the project is 100% ready for the November release of .NET 
> 7.0



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to