Skip to content

Fix getAssetsIndex task for FG1.0#1075

Open
zeng-github01 wants to merge 2 commits into
MinecraftForge:FG_1.0from
zeng-github01:FG_1.0-assets-download
Open

Fix getAssetsIndex task for FG1.0#1075
zeng-github01 wants to merge 2 commits into
MinecraftForge:FG_1.0from
zeng-github01:FG_1.0-assets-download

Conversation

@zeng-github01
Copy link
Copy Markdown

@zeng-github01 zeng-github01 commented May 18, 2026

@LexManos Please review it

@zeng-github01 zeng-github01 force-pushed the FG_1.0-assets-download branch from 954e0eb to 57c9ee0 Compare May 18, 2026 12:31
public static final OperatingSystem OPERATING_SYSTEM = getOs();
public static final SystemArch SYSTEM_ARCH = getArch();
public static final String HASH_FUNC = "MD5";
public static final String USER_AGENT = "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.11 (KHTML, like Gecko) Chrome/23.0.1271.95 Safari/537.11";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do not run code formatters over existing code.
It makes your entire change set a pain to actually read/understand

public static final String FERNFLOWER = "{CACHE_DIR}/minecraft/fernflower.jar";
public static final String EXCEPTOR = "{CACHE_DIR}/minecraft/exceptor.jar";
public static final String ASSETS = "{CACHE_DIR}/minecraft/assets";
public static final String JSONS_DIR = "{CACHE_DIR}/minecraft/versionJsons";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Don't use upper cases in directory names

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants