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

Localises #492

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

DangerRevolution
Copy link
Contributor

@DangerRevolution DangerRevolution commented Jun 29, 2024

Description

Localises da game; opening this PR to run tests on Github and while I do manual checks; I will expand on this section later.

This PR cleans up the salad description; as that causes errors when running a locale script.

This PR cleans up the backpacks.yml folder in the Delta-V back clothing folder; as having a folder with .yml on the end of it caused errors in the script.

Some (many) entities with no description; and no description that can be inherited from a parent instead point to the parent to look for descriptions from that parent. This opens about 80 or so non-critical errors when building.

Needs a comb-through as some "entities" that aren't real entities haven't been localised. Such as "stress test" debug entities.


TODO

  • Initial Locale
  • Checking for errors
  • Discussing whether or not this is worth it

@github-actions github-actions bot added Changes: Localization Changes any ftl files Changes: YML Changes any yml files labels Jun 29, 2024
@DangerRevolution DangerRevolution added Priority: 5-Whenever Doesn't matter when this gets resolved Status: Do Not Merge Do not merge Status: Needs Discussion Must be discussed Size: 1-Very Large For especially large issues/PRs Type: Feature Creation of or significant changes to a feature and removed Changes: YML Changes any yml files labels Jun 29, 2024
@DangerRevolution DangerRevolution self-assigned this Jun 29, 2024
@DangerRevolution DangerRevolution marked this pull request as draft June 29, 2024 15:02
@DangerRevolution
Copy link
Contributor Author

Setting to draft; it looks like I'm going to have to clean up a fair amount of the base entities that were localised.

@Mnemotechnician
Copy link
Contributor

Mnemotechnician commented Jun 29, 2024

AFAIK entity names & descriptions are usually kept in-line so forks in other languages can easily localize them without modifying upstream code, this may disrupt the regular workflow.

Not to mention having 1000 more files will slow down loading and make it harder to introduce changes without actually improving anything else...

@DangerRevolution
Copy link
Contributor Author

AFAIK entity names & descriptions are usually kept in-line so forks in other languages can easily localize them without modifying upstream code, this may disrupt the regular workflow.

Not sure what you mean by this?

@DangerRevolution DangerRevolution marked this pull request as ready for review July 6, 2024 19:35
@github-actions github-actions bot added the Changes: YML Changes any yml files label Jul 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Changes any ftl files Changes: YML Changes any yml files Holy Shit Priority: 5-Whenever Doesn't matter when this gets resolved Size: 1-Very Large For especially large issues/PRs Status: Do Not Merge Do not merge Status: Needs Discussion Must be discussed Type: Feature Creation of or significant changes to a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants