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

~~~ SEMI-FREEZE UNDERWAY; READ DESCRIPTION ~~~ #16061

Closed
duncathan opened this issue Mar 12, 2016 · 9 comments · Fixed by #16026
Closed

~~~ SEMI-FREEZE UNDERWAY; READ DESCRIPTION ~~~ #16061

duncathan opened this issue Mar 12, 2016 · 9 comments · Fixed by #16026
Assignees
Labels
Mapping This map needs better tile-placing. Issue only label.

Comments

@duncathan
Copy link
Contributor

In #16026 I'm about to begin my efforts of repathing every turf in the code. Because of the magnitude of this change, I'm effecting a semi freeze (with Cheri's permission) to save me from the hellhole of conflicts that would otherwise arise.

  • No new changes to existing maps (open prs are unaffected) will be accepted.
    • Entirely new maps will be accepted, but further changes beyond the initial pr will not.
  • Changes to existing turfs may or may not be accepted, at my discretion.
    • Any path changes to turfs (including additions) will not be accepted.

Sorry if this inconveniences anyone, but there's just too much for me to change without effecting this.

@PKPenguin321
Copy link
Contributor

we should have had something like this for dismemberment

@duncathan
Copy link
Contributor Author

Let bygones be bygones

@Dorsisdwarf Dorsisdwarf added Mapping This map needs better tile-placing. Issue only label. Map Edit labels Mar 13, 2016
@optimumtact
Copy link
Member

Why don't you just write some python code to rewrite maps for you? Is it not a pretty basic path change with some vars ?

I did the same for my AI turrets change to maintain sanity

@Aranclanos
Copy link
Contributor

MY LEGACY

@duncathan
Copy link
Contributor Author

If not for aran none of this would be possible

@KazeEspada
Copy link
Contributor

Soon.tm

@duncathan
Copy link
Contributor Author

It's coming don't worry
I'll get a bunch of work done next week cuz I've got the whole week off from school

@duncathan
Copy link
Contributor Author

Alright, since this is approaching completion I'm tightening up the rules a bit:

NO map changes will be accepted now; not even additions.

@PKPenguin321
Copy link
Contributor

its over...............................

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mapping This map needs better tile-placing. Issue only label.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants