Squirrel/Squirrel.Windows

VersionExtensions Regex does not support package ID ending with numbers

Open

#472 opened on 2015年10月27日

GitHub で見る
 (9 comments) (0 reactions) (0 assignees)C++ (7,025 stars) (1,061 forks)batch import
bug:failing-test-would-be-nicehelp wanted

説明

Hi,

if you use NuGet Package Explorer and your nuget package id ends with a number updates won't work anymore (caused by changes in VersionExtensions.cs for prerelease support).

The NuGet Package Explorer uses a point to seperate the package id from the version number (at least if you don't change the filename).

ID: MyId123 Version: 1.1.0

The result would be "MyId123.1.1.0.nupkg"

The suffix regex will remove ".nupkg" and the version regex will match "123.1.1.0" instead of "1.1.0".

A workaround would be renaming your NuGet Package to "MyId123-1.1.0.nupkg". The code should be changed or this naming issue should be noted somewhere.

コントリビューターガイド

技術スタック
csharp
領域
desktoprelease
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
C# regexNuGet package namingVersionExtensions.csbasic Squirrel.Windows knowledge
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65
調査方針
The issue is in VersionExtensions.cs where the regex for extracting the version from the filename assumes the package ID does not end with digits. Examine the current regex pattern and modify it to correctly separate the ID and version, possibly by using a delimiter like a hyphen or by anchoring the version pattern. The workaround is to rename packages with a hyphen, but the fix should handle both formats. Check the comments for any further context.