Add new release artifact x86_64-unknown-linux-musl

[andros21]

Hi, thanks for stylua formatter! I use it a lot within neovim.

The aim of this PR is to support distro like Alpine Linux (on amd64 arch),
building and attaching a new artifact for the target x86_64-unknown-linux-musl
when releasing with release.yml workflow

From local tests, the build process for this new target finished without errors ... only one warning

 warning: dropping unsupported crate type `cdylib` for target `x86_64-unknown-linux-musl`

related to the [lib] part of Cargo.toml

Let me know if I need to do something ... maybe trigger some CI to test stylua works also for this new target

[JohnnyMorganz]

Sorry for the delay, and thanks for the contribution!

The only thing I am worried about here is the format of the name. Unfortunately we didn't leave much flexibility in the way the artifact name is structured, and I know many tools hardcode the structure to determine what asset to download. We do this ourselves in multiple places:

• VSCode extension: https://github.com/JohnnyMorganz/StyLua/blob/5de9a1da8a8dc628b98240e12ffe24c7fdbbea6e/stylua-vscode/src/util.ts#L18C1-L52
• GitHub Action: https://github.com/JohnnyMorganz/stylua-action/blob/b703a90701bbd044c75258647f5aecd8ddb504e1/src/stylua.ts#L41-L76
• Our NPM package:

  StyLua/stylua-npm-bin/binary.js

  Lines 13 to 39 in 5de9a1d

  	const SUPPORTED_PLATFORMS = [
  	{
  	platform: "win32",
  	arch: "x64",
  	name: "stylua-windows-x86_64",
  	},
  	{
  	platform: "darwin",
  	arch: "x64",
  	name: "stylua-macos-x86_64",
  	},
  	{
  	platform: "darwin",
  	arch: "arm64",
  	name: "stylua-macos-aarch64",
  	},
  	{
  	platform: "linux",
  	arch: "x64",
  	name: "stylua-linux-x86_64",
  	},
  	{
  	platform: "linux",
  	arch: "arm64",
  	name: "stylua-linux-aarch64",
  	},
  	];

• The precommit hook installer:

  StyLua/pyproject.toml

  Lines 5 to 15 in 5de9a1d

  	[tool.release-gitter]
  	git-url = "https://github.com/JohnnyMorganz/StyLua"
  	extract-files = ["stylua"]
  	format = "stylua-{system}-{arch}.zip"
  	exec = "chmod +x stylua"
  	[tool.release-gitter.map-arch]
  	arm64 = "aarch64"
  	[tool.release-gitter.map-system]
  	Darwin = "macos"
  	Windows = "windows"
  	Linux = "linux"

And we will need to make sure this new package name doesn't affect those at the minimum, let alone the third party packagers (but it is impossible to control all of them).

But..., the concern here is only that these tools don't accidently pick up the new artifact, and just use the current ones still. So, we just need to structure it in that way. I think maybe stylua-linux-x86_64-musl might be better instead to not hit a few of those regexes.

[andros21]

Fully agree. I was not aware of this jungle of regexes 😅
Now I change it

What do you think instead to move the alias artifact-alias: stylua-linux now linked with linux-gnu target to the new linux-musl target? It should be more portable on linux platforms cause statically compiled against musl-libc, a minimum version of glibc is not needed

[JohnnyMorganz]

I think we should leave the alias for now. The alias artifact is actually deprecated and I intend to remove it at the next major release.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5de9a1d) 97.02% compared to head (92864c2) 97.03%.
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #834   +/-   ##
=======================================
  Coverage   97.02%   97.03%           
=======================================
  Files          16       16           
  Lines        5989     5995    +6     
=======================================
+ Hits         5811     5817    +6     
  Misses        178      178           

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

[JohnnyMorganz]

Thank you! I don't anticipate any problems, but let's see at the next release. If something comes up, I can just remove the zip from the release artifacts temporarily.