Jordan Dominion

Results 45 comments of Jordan Dominion

Another motivator for the bump: https://www.npmjs.com/advisories/725

Deployment.Environment would be useful to have. My use case is the user can rename an enviroment and I want to reflect it in the API.

What's the problem with this currently?

Might be tricky, remember these just evaluate to `1`/`0`. Maybe a more generic lint that return values of tagged function are "checked."

This definitely shouldn't be default behavior though. There are many functions that return a value that doesn't necessarily need to be checked.

I'm guessing something like ```dm var/list/L = list("1", "0", "0", "1", "1", "0", "1") for("1" in L) //runs 4 times ```

@SpaceManiac is that `for(1 to X)` issue a separate issue orrrr?

I am also running into this issue. See this run: https://github.com/tgstation/tgstation-server/actions/runs/3074603866 Check the artifact `linux-integration-test-coverage-Release-System-MariaDB`. Although this runs code from it, the web assembly Tgstation.Server.Host is not included

Here's a copy of my data collector log. This is with both `PreserveCompilationContext` and `CopyLocalLockFileAssemblies` set to true. [test.datacollector.22-09-19_10-47-46_42281_1.log](https://github.com/coverlet-coverage/coverlet/files/9601020/test.datacollector.22-09-19_10-47-46_42281_1.log) ``` TpTrace Warning: 0 : 20400, 1, 2022/09/19, 10:47:52.529, 1792507324109, datacollector.dll,...