Commit Graph
6 Commits
Author SHA1 Message Date
SyncIt21andGitHub 2c76673f70 Updates Biome & Prettier with linting applied (#96394)
## About The Pull Request
- Update Biome to 2.4.16
- Update prettier to 3.8.3
- Apply biome linting rules to a bunch of files
- Convert some files from common-js to es module

## Changelog
N/A
2026-06-10 09:08:43 +12:00
JeremiahandGitHub 9db2f6916b Sets prettier to run on the repo (#91379)
## About The Pull Request
Prettier (an auto formatter) is set to only run within the tgui folder
currently. This removes that limitation, allowing it to automatically
format all supported files in the repo (.js, .html, .yml
[etc](https://prettier.io/docs/))

I made a few exceptions for bundled and generated files
## Why It's Good For The Game
I'm of the opinion that code should look uniform and am lazy enough to
want CTRL-S to format files without having to think beyond that
## Changelog
2025-05-29 21:23:59 -07:00
9885ce35ef fixes the guide comment remover [no gbp] (#90866)
Co-authored-by: harryob <55142896+harryob@users.noreply.github.com>
2025-04-28 11:01:19 +02:00
1b40c31e13 guide comment remover stops stealing my edits (#90820)
## About The Pull Request
the guide comment remover no longer steals your edits made between the
job being created and being run

## Why It's Good For The Game
evil techno goblins stealing my edits
fixes #89970 

## Changelog
no player facing changes

---------

Co-authored-by: harryob <55142896+harryob@users.noreply.github.com>
2025-04-25 14:26:46 +00:00
san7890andGitHub cf59b0bba1 Null-checks PR Body in removeGuideComments (#71043)
This is what #71041 was supposed to be.

## About The Pull Request
Hey there,

I was looking at #71028, and I noticed that this workflow failed because
oranges left the PR body blank. I think that's silly, so let's just
include an early return so the whole thing doesn't throw.

This is the error:


![image](https://user-images.githubusercontent.com/34697715/199852746-24b2ed53-442d-4c40-b81d-902236183328.png)
via:
https://github.com/tgstation/tgstation/actions/runs/3382875608/jobs/5618241641

Here's an example PR where I fixed it (on my fork, just expand the
"Remove guide comments" step):
https://github.com/san7890/bruhstation/actions/runs/3389994395/jobs/5633658300

## Why It's Good For The Game

It hurts my heart to see checks red out in trivial situations like this.
## Changelog
Literally nothing here concerns players.

Ignore the commit history I was testing this PR the wrong way and was
effectively gaslighting myself (we're good now though)
2022-11-04 01:26:43 -07:00
MothblocksandGitHub acd034f967 Remove guide comments action (#70739)
Removes all default comments after a PR is opened (and only when it is opened).

Means that I can turn on the pr body merge option without keeping all the cruft
2022-10-30 01:46:48 -07:00