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

"rows" is defaulted to just spaces #72699

Merged
merged 10 commits into from
Apr 4, 2024

Conversation

Procyonae
Copy link
Contributor

@Procyonae Procyonae commented Mar 29, 2024

Summary

None

Purpose of change

Fixes #72511

Describe the solution

If "rows" isn't provided a row of the expected size of just spaces is used every time one is expected
Removes existing blank "rows" no longer needed

Describe alternatives you've considered

Testing

Compiles and seems to work

Additional context

Most of the c++ change shown is just whitespace idk why git is highlighting the whole thing

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) labels Mar 29, 2024
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Mar 29, 2024
@GuardianDll
Copy link
Member

could you remove this thing also?

"rows": [
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" ",
" "
],

@Procyonae Procyonae marked this pull request as draft March 29, 2024 21:59
@github-actions github-actions bot added Missions Quests and missions Code: Tests Measurement, self-control, statistics, balancing. Monsters Monsters both friendly and unfriendly. Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. Mods: Dinomod Anything to do with the Dinoclysm mod (DinoMod) Mods: Xedra Evolved Anything to do with Xedra Evolved Mods: Mind Over Matter Mods: Sky Island Anything about the Sky Island mod labels Mar 29, 2024
@Procyonae Procyonae changed the title "palettes" no longer relies on "rows" being defined "rows" is defaulted to just spaces Mar 29, 2024
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Mar 29, 2024
@github-actions github-actions bot removed the BasicBuildPassed This PR builds correctly, label assigned by github actions label Mar 29, 2024
@Procyonae Procyonae marked this pull request as ready for review March 29, 2024 23:05
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Auto-requesting reviews from non-collaborators: @LyleSY @Standing-Storm

@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. [Markdown] Markdown issues and PRs BasicBuildPassed This PR builds correctly, label assigned by github actions labels Mar 29, 2024
@Maleclypse
Copy link
Member

I think Flexbuffers is an akrieger word

src/mapgen.cpp Outdated Show resolved Hide resolved
Excessive amount of errors just because otherwise you get a gibberish one
@github-actions github-actions bot removed the BasicBuildPassed This PR builds correctly, label assigned by github actions label Apr 2, 2024
src/mapgen.cpp Outdated Show resolved Hide resolved
src/mapgen.cpp Outdated Show resolved Hide resolved
src/mapgen.cpp Outdated Show resolved Hide resolved
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Apr 3, 2024
@I-am-Erk
Copy link
Member

I-am-Erk commented Apr 4, 2024

Oh, nice

@I-am-Erk I-am-Erk merged commit e26a862 into CleverRaven:master Apr 4, 2024
23 of 27 checks passed
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 <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing. <Documentation> Design documents, internal info, guides and help. Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display [Markdown] Markdown issues and PRs Missions Quests and missions Mods: Dinomod Anything to do with the Dinoclysm mod (DinoMod) Mods: Mind Over Matter Mods: Sky Island Anything about the Sky Island mod Mods: Xedra Evolved Anything to do with Xedra Evolved Monsters Monsters both friendly and unfriendly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Palettes containing parameters shouldn't be skipped regardless of whether rows is specified
5 participants