-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
New package: GorrilaDevs.Ferium version 4.5.0 #137336
New package: GorrilaDevs.Ferium version 4.5.0 #137336
Conversation
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run |
Automatic Validation ended with:
(Automated response - build 684.) |
Azure Pipelines successfully started running 1 pipeline(s). |
For moderatorsThis account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification. Important Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.) 良い一日をお過ごしください! |
Hi @Exorcism0666, The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow. (Automated response - build 691.) |
Automatic Validation ended with:
(Automated response - build 691.) |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
Hi @Exorcism0666, The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow. (Automated response - build 694.) |
Automatic Validation ended with:
(Automated response - build 694.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Exorcism0666 just some minor nitpicks, otherwise I approve.
LGTM 👍
ReleaseDate: 2024-01-30 | ||
Installers: | ||
- Architecture: x64 | ||
InstallerUrl: https://github.com/gorilla-devs/ferium/releases/download/v4.5.0/ferium-windows-msvc.zip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to use the PackageVersion
variable here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you think about future updates, the action I have linked (https://github.com/vedantmgoyal2009/winget-releaser) is able to handle that as well.
- RelativeFilePath: ferium.exe | ||
Dependencies: | ||
PackageDependencies: | ||
- PackageIdentifier: Microsoft.VCRedist.2015+.x64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the VS runtime actually required? If so, could I know how you determined the runtime dependencies of the executable?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me this works without that, but I might have a different version for it from one of the games I have installed.
License: MPL-2.0 | ||
LicenseUrl: https://github.com/gorilla-devs/ferium/blob/main/LICENSE.txt | ||
ShortDescription: Ferium is a fast and feature rich CLI program for downloading and updating Minecraft mods from Modrinth, CurseForge, and GitHub Releases, and modpacks from Modrinth and CurseForge. | ||
ReleaseNotes: |- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If markdown is supported, could we replace this with the actual changelog contents:
- Added aliases for many commands and arguments
- The `--version`/`-v` flags only work for the top-level command now (they do not propagate to subcommands)
- Renamed `--dont-check-game-version` and `--dont-check-mod-loader` to `--ignore-game-version` and `--ignore-mod-loader` respectively
- Also added a short hand `-V` and `-M`
- The previous flags still work as a hidden alias, so scripts do not have to be edited
- Added `ferium profile info` and `ferium modpack info` subcommands to print information about the current profile/modpack
- These are aliased to `ferium profile` and `ferium modpack`
- Added `ferium profiles` and `ferium modpacks`, which are aliases to their respective list commands
- Added option to temporarily ignore game version and mod loader checks and force-add the mod anyways ([#142](https://github.com/gorilla-devs/ferium/issues/142))
- Added an argument for providing a profile/modpack to switch to after deleting a profile/modpack ([#390](https://github.com/gorilla-devs/ferium/issues/390))
- Fixed errors not getting caught when adding mods (e.g. `HTTP error 404` instead of `mod does not exist`)
- Moved a majority of the mod adding code to libium
- Added a header with profile information in `ferium list`
- Made `ferium list` display the source and mod ID first and mod name last to circumvent minor formatting issues
- Added user agent to GitHub API check to make it more reliable
- Added more information and coloured formatting to profile and modpack pickers (when deleting or switching)
- Made the remove picker show the same information as `ferium list` but without colours
- Fixed distribution denied errors not getting caught during mod resolution and causing the entire command to fail instead ([#308](https://github.com/gorilla-devs/ferium/issues/308))
# Created with Komac v1.11.0 | ||
# yaml-language-server: $schema=https://aka.ms/winget-manifest.installer.1.5.0.schema.json | ||
|
||
PackageIdentifier: GorrilaDevs.Ferium |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this correct?
PackageIdentifier: GorrilaDevs.Ferium | |
PackageIdentifier: GorillaDevs.Ferium |
See also: https://github.com/gorilla-devs/ferium
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yes that is indeed a typo
Other than that the manifest works as expected for me: where ferium
# %LOCALAPPDATA%/Microsoft/WinGet/Packages/GorrilaDevs.Ferium__DefaultSource/ferium |
See #137328 (comment), @Exorcism0666. Specifically, I somehow misspelt “Gorilla”. |
Also I'm not sure what is up Defender flagging |
@stephengillie @ST-DDT @RokeJulianLockhart I will reopen a pull request because of the typo :c |
Pull request has been created with Komac v1.11.0 🚀
GorillaDevs.Ferium
#137328Microsoft Reviewers: Open in CodeFlow