Skip to content
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

license extension rename broke links #777

Closed
3cpt opened this issue Feb 22, 2021 · 5 comments · Fixed by #781
Closed

license extension rename broke links #777

3cpt opened this issue Feb 22, 2021 · 5 comments · Fixed by #781

Comments

@3cpt
Copy link

3cpt commented Feb 22, 2021

Description

IMPORTANT: if the defect is reproduced only in a workflow from within the Visual Studio IDE then do not report the issue here - instead, please report it using Visual Studio's "Send Feedback" option that can be accessed from the Help menu OR using this link https://developercommunity.visualstudio.com.

For a defect specific to the MSTest V2 test framework, describe the issue you've observed.

Side effect generated by #603. Also, taking in consideration that Github support LICENSE, why add the .txt?
I can open an MR, but in my opinion the problem is the file extension. Waiting for feedback.

Steps to reproduce

  • Go to the License section on README.md and click on the MIT license link

Expected behavior

Open license content

Actual behavior

404

Haplois added a commit that referenced this issue Feb 22, 2021
@Haplois
Copy link
Contributor

Haplois commented Feb 22, 2021

Thank you for the feedback. Let's remove the extension to comply with GitHub. I started a PR and abandoned it; if we're removing the extension, we need to update the build script and all other references to it.

Let me know if you like to do the PR, or I can do it as well.

@3cpt
Copy link
Author

3cpt commented Feb 22, 2021

I can do the PR, no problem.

@Haplois
Copy link
Contributor

Haplois commented Feb 23, 2021

Thank you, please @ me after you've created with the PR.

@jnyrup
Copy link
Contributor

jnyrup commented Mar 19, 2021

Nuget seems to have problems linking to extension-less license files

image

Scenario: I use Get-Package | Select-Object Id,LicenseUrl to list all licenses in my projects and for MSTest.TestAdapter it prints

MSTest.TestAdapter                                              https://aka.ms/deprecateLicenseUrl

As far as I understand the license displayed on show on https://www.nuget.org/packages/MSTest.TestAdapter/
is generated upon ingestion of the nuget package where it stores the license file from <PackageLicenseFile> and generates a valid link to it.

If this is changed to

<PackageLicenseExpression>MIT</PackageLicenseExpression>

the nuget pack will include https://licenses.nuget.org/MIT in the generated .nuspec file.

Optionally use a combined approach, using <PackageLicenseExpression> and embedding the license file, see https://github.com/JamesNK/Newtonsoft.Json/blob/master/Src/Newtonsoft.Json/Newtonsoft.Json.csproj

@Haplois
Copy link
Contributor

Haplois commented Mar 22, 2021

Hi,

It's not about extensionless license files. The issue you're having is because licenseUrl is depracated. Even if we update this, you should stop using that and use the license expression included with the package itself.

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 a pull request may close this issue.

3 participants