-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
eac3to@3.50: update from 3.36 #5501
Conversation
coolpringles
commented
Feb 5, 2024
- I have read the Contributing Guide.
/verify |
Your changes do not pass checks. eac3to
|
/verify |
Your changes do not pass checks. eac3to
|
/verify |
All changes look good. Wait for review from human collaborators. eac3to
|
I did not mean to do that |
/verify |
Your changes do not pass checks. eac3to
|
/verify |
All changes look good. Wait for review from human collaborators. eac3to
|
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.
Hi, thanks for this update, could you please:
- restore autoupdate (point to the same url as checkver but add on the end
https://www.rationalqm.us/eac3to/eac3to_$version.rar
- reorder the manifest to the regular order here: https://github.com/ScoopInstaller/.github/blob/main/.github/CONTRIBUTING.md#for-scoop-buckets
- restore the same number of spaces for indentation as standard for all manifests (4) - this is set in
.editorconfig
if your editor checks that
/verify |
Your changes do not pass checks. eac3to
|
/verify |
All changes look good. Wait for review from human collaborators. eac3to
|
Thanks for your contribution! For reference, this PR updates the download, checkver, and autoupdate to www.rationalqm.us as it says on the old forum post:
|