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

Update the readme a tad. #13228

Merged
merged 2 commits into from Jan 3, 2017

Conversation

PJB3005
Copy link
Contributor

@PJB3005 PJB3005 commented Jan 2, 2017

Travis-CI and a Discord link.

@PJB3005 PJB3005 changed the title Adds Travis-CI status to the readme. Update the readme a tad. Jan 2, 2017
@Probe1 Probe1 added the Logging / Administration This touches things involving admins or logging. label Jan 2, 2017
@Probe1 Probe1 merged commit 43feddf into vgstation-coders:Bleeding-Edge Jan 3, 2017
@gbasood
Copy link
Contributor

gbasood commented Jan 3, 2017

next up code coverage @PJB3005 :^)

@PJB3005 PJB3005 deleted the travis-ci-status branch January 3, 2017 20:54
@PJB3005
Copy link
Contributor Author

PJB3005 commented Jan 3, 2017

unit tests

byond

@ComicIronic
Copy link
Contributor

Actually, what would be doable for unit testing? Spinning up a specially-config'd world which runs a bunch of code in the tests module under test datums which isn't otherwise included?

@gbasood
Copy link
Contributor

gbasood commented Jan 6, 2017

it'd be really hard for us to do proper unit testing
for simple things, maybe machines, you could feed them dummy data and see if they don't runtime and that's probably about it

wouldn't even need to spin up a full world

ihadtoregisterforthis pushed a commit to ihadtoregisterforthis/fork4 that referenced this pull request Jul 3, 2017
* Adds Travis-CI status to the readme.

* Hey add a Discord link in there too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Logging / Administration This touches things involving admins or logging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants