Skip to content

small upgrades of logging for GTNH - #4304

Merged
itzg merged 1 commit into
itzg:masterfrom
IronMine:master
Oct 1, 2026
Merged

itzg merged 1 commit into
itzg:masterfrom
IronMine:master

Conversation

@IronMine

Copy link
Copy Markdown
Contributor

Not sure if this will be of any help for someone but i did it anyway !

GTNH_PACK_VERSION=latest-beta.
-BEFORE-
Selected "2.9.0-RC-1" as latest beta version for download.

-AFTER- NO CHANGE
Selected "2.9.0-RC-1" as latest beta version for download.

GTNH_PACK_VERSION=latest.
-BEFORE-
Selected "2.8.4" as latest version for download.

-AFTER- NO CHANGE
Selected "2.8.4" as latest version for download.

GTNH_PACK_VERSION=2.9.0-RC-1.
-BEFORE-
Selected 2.9.0-RC-1 as matching version for download.

-AFTER- Brackets
Selected "2.9.0-RC-1" as matching version for download.

GTNH_PACK_VERSION=2.10.0.
-BEFORE-
Selected as matching version for download.
[ERROR] No release found for GTNH_PACK_VERSION=2.10.0. Please check if the version is valid.

-AFTER- No log before error
[ERROR] No release found for GTNH_PACK_VERSION=2.10.0. Please check if the version is valid.

gtnh_versions_json_url="https://downloads.gtnewhorizons.com/versions.j".
-BEFORE-
curl: (22) The requested URL returned error: 404
Selected as latest beta version for download.
[ERROR] No release found for GTNH_PACK_VERSION=latest-beta. Please check if the version is valid.

-AFTER- No log before error and better explaination
curl: (22) The requested URL returned error: 404
[ERROR] Failed to download from "https://downloads.gtnewhorizons.com/versions.j"

@itzg itzg left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logs do look better. And nice refactoring.

@itzg
itzg merged commit 67d1453 into itzg:master Oct 1, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants