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

[Solaris Ridge] Aesthetic Update #6152

Closed
wants to merge 10 commits into from

Conversation

Steelpoint
Copy link
Contributor

@Steelpoint Steelpoint commented Apr 15, 2024

About the pull request

Effects a significant aesthetic makeover over parts of Solaris Ridge.

This PR focuses more on the Northern half of the colony.

(please use MapDiffBot2 to view changes)

Explain why it's good for the game

I feel a long term goal of mapping should be to enhance our older maps to make them look far more "alive", lived in and aesthetically pleasing. But keeping the spirit of the similar layout of the original map.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
maptweak: A significant amount of aesthetic upgrades to Solaris Ridge have been added.
/:cl:

@github-actions github-actions bot added the Mapping did you remember to save in tgm format? label Apr 15, 2024
@Steelpoint Steelpoint marked this pull request as draft April 15, 2024 17:12
@Steelpoint
Copy link
Contributor Author

This project might take longer since I'm going to be doing more of the colony up.

@Steelpoint Steelpoint changed the title [Solaris Ridge] Aesthetic Mapping : Roads And Shrubs [Solaris Ridge] Aesthetic Update Apr 17, 2024
@Steelpoint Steelpoint marked this pull request as ready for review April 17, 2024 11:29
@Steelpoint
Copy link
Contributor Author

I'll work on the southern half of the colony in a separate PR but I think this is a good place to work from with what I've done thus far.

Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Apr 25, 2024
@Drulikar Drulikar removed the Stale beg a maintainer to review your PR label Apr 30, 2024
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label May 1, 2024
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Copy link
Contributor

github-actions bot commented May 9, 2024

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label May 9, 2024
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label May 12, 2024
@Steelpoint
Copy link
Contributor Author

Likely going to have to re-do the PR since I can't solve map merge conflicts to save my life.

@realforest2001 realforest2001 marked this pull request as draft May 12, 2024 16:01
@Nanu308 Nanu308 removed the Stale beg a maintainer to review your PR label May 12, 2024
@Steelpoint Steelpoint closed this May 13, 2024
@Steelpoint Steelpoint reopened this Jul 1, 2024
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Jul 1, 2024
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Jul 1, 2024
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@Steelpoint Steelpoint closed this Jul 1, 2024
@Steelpoint Steelpoint deleted the NewKutjevo branch July 17, 2024 17:00
github-merge-queue bot pushed a commit that referenced this pull request Sep 1, 2024
# About the pull request

Continuation of this PR: #6152

--

Effects a significant aesthetic makeover over parts of Solaris Ridge.

This PR focuses more on the Northern half of the colony. I intend to
followup in the future with doing more work on the southern half and the
caves.

--

# Explain why it's good for the game

I feel a long term goal of mapping should be to enhance our older maps
to make them look far more "alive", lived in and aesthetically pleasing.
But keeping the spirit of the similar layout of the original map.


# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

Put screenshots and videos here with an empty line between the
screenshots and the `<details>` tags.

</details>


# Changelog
:cl:
maptweak: The northern half of the main colony of Solaris Ridge has
received an aesthetic upgrade.

/:cl:

---------

Co-authored-by: Steelpoint <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mapping did you remember to save in tgm format?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants