Skip to content
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

docs: clarify logoFile filename restrictions #174

Closed
wants to merge 1 commit into from

Conversation

mainrs
Copy link

@mainrs mainrs commented Oct 6, 2024

Closes #157.

| `version` | string | `"1"` | The version of the mod, preferably in a [variation of Maven versioning][versioning]. When set to `${file.jarVersion}`, it will be replaced with the value of the `Implementation-Version` property in the JAR's manifest (displays as `0.0NONE` in a development environment). | `version="1.20.2-1.0.0"` |
| `displayName` | string | value of `modId` | The display name of the mod. Used when representing the mod on a screen (e.g., mod list, mod mismatch). | `displayName="Example Mod"` |
| `description` | string | `'''MISSING DESCRIPTION'''` | The description of the mod shown in the mod list screen. It is recommended to use a [multiline literal string][multiline]. This value is also translatable, see [Translating Mod Metadata][i18n] for more info. | `description='''This is an example.'''` |
| `logoFile` | string | _nothing_ | The name and extension of an image file used on the mods list screen. The logo must be in the root of the JAR or directly in the root of the source set (e.g. `src/main/resources` for the main source set). Valid filename characters are lowercase letters (`a-z`), digits (`0-9`), underscores (`_`), periods (`.`) and hyphens (`-`). The complete character set is `[a-z0-9_-.]`. | `logoFile="example_logo.png"` |
Copy link
Contributor

Choose a reason for hiding this comment

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

If we want to be more accurate, it would be good to specify that the image can be in a subdirectory of the root of the JAR or source set, as long as the path is absolute.

For example test/example_logo.png would be a valid asset location if at src/main/resources/test/example_logo.png. However using ./test/example_logo.png or ../test/example_logo.png would not. This can be added for the other versions as well as this change occurred in 1.20.2.

Copy link
Author

Choose a reason for hiding this comment

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

How can I do that retroactively for other Versions as well?

Copy link
Contributor

Choose a reason for hiding this comment

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

All versions are within the same branch, just in the versioned_docs, so you can do an across project search in your editor of choice for the word logoFile. That'll probably be the easiest.

@IchHabeHunger54 IchHabeHunger54 added the addition Adding or rewriting information. label Oct 7, 2024
ChampionAsh5357 added a commit to ChampionAsh5357/NeoForge-Docs that referenced this pull request Oct 14, 2024
@ChampionAsh5357 ChampionAsh5357 mentioned this pull request Oct 17, 2024
20 tasks
@neoforged-automation
Copy link

@mainrs, this pull request has conflicts, please resolve them for this PR to move forward.

@neoforged-automation neoforged-automation bot added the needs rebase This Pull Request needs to be rebased before being merged label Nov 15, 2024
@ChampionAsh5357
Copy link
Contributor

Added by #178

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
addition Adding or rewriting information. needs rebase This Pull Request needs to be rebased before being merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document valid characters for logoFile in neoforge.mods.toml
3 participants