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

Fish size cleanup #66548

Merged
merged 6 commits into from
Jul 1, 2023
Merged

Conversation

Karol1223
Copy link
Contributor

@Karol1223 Karol1223 commented Jun 30, 2023

Summary

None

Purpose of change

In #61614 I audited fish sizes. And like an absolute moron, I calculated them as CUBES. We all know fish are cylinders and not cubes. This is unacceptable. My immersion is ruined.

Describe the solution

Recalculated all the fish as cylinders and not cubes - most now have lower volume, except a few where I miscalculated it in the original PR. A few have changed weights since back then I didn't pick out the correct average weights. Pickerel renamed to Chain pickerel since I forgot to do it the last time.

Also apparently threw the dictionary through a tool to make it alphabetical. Smh people.

Describe alternatives you've considered

Testing

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Jun 30, 2023
@github-actions
Copy link
Contributor

Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details.

Click to expand
  • chain pickerels

This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to tools/spell_checker/dictionary.txt so they will not trigger an alert next time.

@Procyonae
Copy link
Contributor

They should be two back to back cones smh /s

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jul 1, 2023
@Maleclypse Maleclypse merged commit 030a610 into CleverRaven:master Jul 1, 2023
24 checks passed
@Karol1223 Karol1223 deleted the le-fishe-au-chocolate branch July 1, 2023 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants