Dzoukr / Dapper.FSharp

Lightweight F# extension for StackOverflow Dapper with support for MSSQL, MySQL, PostgreSQL, and SQLite
MIT License
365 stars 35 forks source link

Do not force specific SDK version 3.1.409 #33

Closed prosconi closed 3 years ago

prosconi commented 3 years ago

Self explanatory

https://github.com/Dzoukr/Dapper.FSharp/blob/e864e07d8be727633a761e3caf16b61e554956eb/global.json#L1-L5

Dzoukr commented 3 years ago

What's the problem with it? It's for local development and can be deleted if you don't like it.

prosconi commented 3 years ago

This file is in source control so the process for people who do not have that specific SDK would be to:

either

or install that specific SDK version

Dzoukr commented 3 years ago

or install that specific SDK version

Yup. Exactly. And this is how it should be. global.json is like a paket.lock - a well-known definition of expected dependencies for the correct (repeatable) build. From a CI perspective, it's a must-have thing (you don't want the local build environment to affect the build output). There can be so many dragons between SDK versions (like push to Nuget feed) and if you want to be safe, use the locked version (https://docs.microsoft.com/en-us/dotnet/core/tools/global-json).

In other words: The project is responsible for setting expectations. If you want to contribute, you can:

But the final version will be built/pushed using locked SDK anyway. 🤷‍♀️