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

[WIP] Example buildings to be JSONized #22366

Merged
merged 2 commits into from Oct 3, 2018

Conversation

5 participants
@SunshineDistillery
Copy link
Contributor

commented Nov 6, 2017

SUMMARY: Mods "Even more buildings."

These are some example buildings for #21824. It isn't meant to be merged as is; it's in mod form so it's easy to load up and examine the sites.

Rather than just remake cube buildings, I altered them a bit to make them more interesting/realistic. The Missile Silo currently lacks poison gas, a launch computer, and a missile.

For the real thing, I can condense them to single files or leave them as they are.

}
}
}
]

This comment has been minimized.

Copy link
@Coolthulhu

Coolthulhu Nov 6, 2017

Contributor

Could use styling

@Coolthulhu

This comment has been minimized.

Copy link
Contributor

commented Nov 6, 2017

Missile computer should be jsonable now.

@BorkBorkGoesTheCode

This comment has been minimized.

Copy link
Contributor

commented Jan 15, 2018

Are you here @SunshineDistillery ?

@illi-kun

This comment has been minimized.

Copy link
Member

commented Apr 1, 2018

@ZhilkinSerg it seems that you have some experience with implementation of similar changes and it also seems that this PR is obsolete. It would be really great if you can re-implement this in bunch of smaller PRs (with possible changes, of course, it's totally up to you).

@ZhilkinSerg

This comment has been minimized.

Copy link
Contributor

commented Apr 3, 2018

@ZhilkinSerg it seems that you have some experience with implementation of similar changes and it also seems that this PR is obsolete. It would be really great if you can re-implement this in bunch of smaller PRs (with possible changes, of course, it's totally up to you).

I will pick this up soon.

@mlangsdorf mlangsdorf changed the title Example buildings to be JSONized [WIP] [WIP] Example buildings to be JSONized Sep 11, 2018

@ZhilkinSerg ZhilkinSerg merged commit e3952d7 into CleverRaven:master Oct 3, 2018

0 of 3 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
gorgon-ghprb Build triggered for merge commit.
Details

Mapgen and overmapgen improvements automation moved this from In progress to Done Oct 3, 2018

@ZhilkinSerg ZhilkinSerg removed their assignment Oct 3, 2018

@ZhilkinSerg

This comment has been minimized.

Copy link
Contributor

commented Oct 3, 2018

I've added computers and poison gas and linted whole mod in #26030.

@ZhilkinSerg ZhilkinSerg referenced this pull request Oct 3, 2018

Merged

Fixed mapspecials mod #26030

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.