Skip to content

Commit

Permalink
Merge branch 'ss220club:master' into donor_jobs_update
Browse files Browse the repository at this point in the history
  • Loading branch information
PhantornRU authored Dec 8, 2023
2 parents 52ef249 + 977ea3d commit 21d761d
Show file tree
Hide file tree
Showing 267 changed files with 3,181 additions and 2,161 deletions.
16 changes: 16 additions & 0 deletions .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,22 @@ eg: `/datum/thing`, not `datum/thing`
In DM, this is optional, but omitting it makes finding definitions harder. To be specific, you can declare the path `/arbitrary`, but it
will still be, in actuality, `/datum/arbitrary`. Write your code to reflect this.

### Do not use list operators in strings

The use of list operators to augment strings is not allowed. This is roughly 10 times slower than using a list with a Join() Function.

```dm
//Bad
var/text = "text"
text += "More text"
to_chat(world, text)
//Good
var/list/text = list("text")
text += "More text"
to_chat(world, text.Join(""))
```

### Do not use text/string based type paths

It is rarely allowed to put type paths in a text format, as there are no compile errors if the type path no longer exists. Here is an example:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/auto_translate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ jobs:
- name: 'Push Translation'
run: |
git fetch origin translate
git checkout origin/translate
git config --local user.email "[email protected]"
git config --local user.name "ss220bot"
Expand Down
4 changes: 0 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -108,10 +108,6 @@ jobs:
sudo apt update || true
sudo apt install libssl1.1:i386
ldd librust_g.so
- name: Start Redis
uses: supercharge/[email protected]
with:
redis-version: 6
- name: Compile & Run Unit Tests
run: |
tools/ci/install_byond.sh '${{ matrix.byondtype }}'
Expand Down
6 changes: 2 additions & 4 deletions _maps/map_files/MetaStation/MetaStation.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -55190,13 +55190,12 @@
d2 = 8;
icon_state = "4-8"
},
/obj/effect/mapping_helpers/airlock/access/all/engineering/general,
/obj/effect/mapping_helpers/airlock/access/any/engineering/general,
/obj/machinery/door/airlock/engineering/glass{
heat_proof = 1;
name = "Supermatter Chamber"
},
/obj/effect/mapping_helpers/airlock/access/any/engineering/atmos,
/obj/effect/mapping_helpers/airlock/access/any/engineering/general,
/turf/simulated/floor/engine,
/area/station/engineering/engine/supermatter)
"gvN" = (
Expand Down Expand Up @@ -74829,13 +74828,12 @@
/turf/simulated/floor/plating,
/area/station/maintenance/xenobio_south)
"peF" = (
/obj/effect/mapping_helpers/airlock/access/all/engineering/general,
/obj/effect/mapping_helpers/airlock/access/any/engineering/general,
/obj/machinery/door/airlock/engineering/glass{
heat_proof = 1;
name = "Supermatter Chamber"
},
/obj/effect/mapping_helpers/airlock/access/any/engineering/atmos,
/obj/effect/mapping_helpers/airlock/access/any/engineering/general,
/turf/simulated/floor/engine,
/area/station/engineering/engine/supermatter)
"pfh" = (
Expand Down
6 changes: 3 additions & 3 deletions _maps/map_files/RandomRuins/SpaceRuins/moonoutpost19.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -5151,7 +5151,7 @@
/obj/effect/decal/cleanable/cobweb2,
/obj/effect/turf_decal/delivery/hollow,
/obj/structure/rack,
/obj/item/assembly/signaler/anomaly/random,
/obj/item/paper/researchnotes,
/turf/simulated/floor/plasteel{
icon_state = "dark"
},
Expand Down Expand Up @@ -8441,13 +8441,13 @@
/area/ruin/space/unpowered)
"Fm" = (
/obj/structure/safe,
/obj/item/paper/researchnotes,
/obj/item/clothing/glasses/hud/security/sunglasses/fluff/eyepro,
/obj/machinery/light/small{
dir = 4
},
/obj/effect/decal/cleanable/dirt,
/obj/effect/turf_decal/delivery/hollow,
/obj/item/assembly/signaler/anomaly/random,
/obj/item/storage/lockbox/experimental_weapon,
/turf/simulated/floor/plasteel{
icon_state = "dark"
},
Expand Down
2 changes: 2 additions & 0 deletions _maps/map_files/RandomRuins/SpaceRuins/ussp.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -1075,6 +1075,7 @@
/area/ruin/space/derelict/crew_quarters)
"cK" = (
/obj/structure/table,
/obj/item/soap/deluxe/laundry,
/turf/simulated/floor/plasteel{
dir = 5;
icon_state = "blue"
Expand Down Expand Up @@ -1191,6 +1192,7 @@
"cZ" = (
/obj/structure/table,
/obj/effect/decal/cleanable/dirt,
/obj/item/soap/deluxe/laundry,
/turf/simulated/floor/plasteel{
dir = 1;
icon_state = "blue"
Expand Down
6 changes: 3 additions & 3 deletions _maps/map_files/generic/Lavaland.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -7527,7 +7527,7 @@
dir = 4
},
/obj/effect/mapping_helpers/airlock/access/any/supply/mining,
/obj/effect/mapping_helpers/airlock/access/all/supply/mining_station,
/obj/effect/mapping_helpers/airlock/access/any/supply/mining_station,
/turf/simulated/floor/plasteel{
icon_state = "darkgreencorners"
},
Expand Down Expand Up @@ -8026,11 +8026,11 @@
name = "Shuttle Dock"
},
/obj/machinery/door/firedoor,
/obj/effect/mapping_helpers/airlock/access/any/supply/mining,
/obj/effect/mapping_helpers/airlock/access/all/supply/mining_station,
/obj/effect/mapping_helpers/airlock/unres{
dir = 8
},
/obj/effect/mapping_helpers/airlock/access/any/supply/mining,
/obj/effect/mapping_helpers/airlock/access/any/supply/mining_station,
/turf/simulated/floor/plasteel{
dir = 8;
icon_state = "darkredfull"
Expand Down
Loading

0 comments on commit 21d761d

Please sign in to comment.