Skip to content

Conversation

duckboycool
Copy link
Contributor

What is this fixing or adding?

I noticed that in the messages added in #5478, there were spaces mistakenly missing between the parts of the string that were on different lines, so I tried to check if there were other instances. There were.

How was this tested?

I tried to get a couple of these to print in order to test them, but most of them aren't. I'm pretty sure all of these are unintentional, (aside from maybe the couple of removed ones in CV64).

If this makes graphical changes, please attach screenshots.

🚀🧑‍🚀

@github-actions github-actions bot added affects: core Issues/PRs that touch core and may need additional validation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Oct 17, 2025
"Those cows are weird.",
0x07: "Preserved with special\n"
" wall-based methods.",
"wall-based methods.",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This and the other one in this file are the ones I mentioned being unsure.

@Berserker66
Copy link
Member

The changes to Factorio and LttP look fine to me.

@NewSoupVi NewSoupVi added the waiting-on: world-maintainer Issue/PR is waiting for feedback or approval by the maintainer of a world. label Oct 20, 2025
@NewSoupVi
Copy link
Member

NewSoupVi commented Oct 20, 2025

This kind of thing should really be one PR per world or world maintainer, even if it's a lot of tiny changes. It's annoying I know

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

Labels

affects: core Issues/PRs that touch core and may need additional validation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. waiting-on: world-maintainer Issue/PR is waiting for feedback or approval by the maintainer of a world.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants