Updates to clean up tests

Ignore links to localhost in tests

This org doesn't exist

Removing unmaintained app (returning 503)

Use `/ping` for tests since there is an option to disable stats
This commit is contained in:
tcbyrd
2018-04-06 07:38:53 -04:00
parent 2e7d3cb4d3
commit 14292dc7ea
4 changed files with 2 additions and 62 deletions

View File

@@ -13,7 +13,6 @@ host: https://github-configurer.herokuapp.com
installations: 128
organizations:
- apollographql
- CNXTEoEorg
- denysdovhan
- bkeepers
- chaijs

View File

@@ -1,60 +0,0 @@
---
title: Untrivializer
description: An app that asks if you're really sure something is trivial when you
make a comment in Github issues.
slug: https://github.com/apps/untrivializer
screenshots:
- https://raw.githubusercontent.com/entendu/untrivializer/master/docs/ss.png
authors:
- entendu
repository: entendu/untrivializer
host: https://puffy-aftermath.glitch.me
stars: 5
updated: 2017-12-15 23:13:31 UTC
installations: 3
organizations:
- red-eclipse
- entendu
---
# untrivializer
> a GitHub App built with [probot](https://github.com/probot/probot) that asks in Github comments if you're really sure something is trivial.
![screenshot](https://raw.githubusercontent.com/entendu/untrivializer/master/docs/ss.png)
## But why tho
A lot of times we'll write up an issue without really thinking it through:
> We want _X_. Just do thing _Y_ to get it, that should be really simple.
> Make change _Z_, it should be trivial.
> Do thing _Foo_, it's super easy.
Doing those things (almost without fail) takes way, way more time than we think it will. Why? A couple reasons:
* If we're making a comment like that we're probably not thinking through all the little details anyway,
* We're all just [terrible at estimating, _even when we know we're terrible at estimating_](https://en.wikipedia.org/wiki/Planning_fallacy) (it's truly incredible),
* We often forget the downstream repercussions of "simple" changes -- one line of code for you might mean 3 hours or more of code review, testing, etc.
So this little robot tries to detect when you've said something "should be simple" and asks if that's really the case. The default reply is:
> Did you mean, "_might be straightforward, but could have unforseen complexities that would completely change the prioritization of the issue, so we should let it go through the normal planning and estimation process_"[?](https://en.wikipedia.org/wiki/Planning_fallacy)
Detection is an ugly regex.
## Running
I think you can run this as a Github app here: https://github.com/apps/untrivializer but no guarantees that the backend will stay up. If you'd rather run it yourself (I don't blame you) look below.
## Setup
```
# Install dependencies
npm install
# Run the bot
npm start
```

View File

@@ -9,6 +9,7 @@ url_ignores = [
%r{https://github.com/apps/.*/installations/new},
"https://mochajs.org/",
%r{mit-license\.org}, # misconfigured SSL or old certs on Travis CI?
%r{https://localhost:*/*}, # Ignore links to localhost in documentation
]
begin

View File

@@ -53,7 +53,7 @@ describe "lint test" do
end
it "host returns 200" do
uri = URI(data["host"] + '/probot/stats')
uri = URI(data["host"] + '/ping')
res = Net::HTTP.get_response(uri)
assert_equal "200", res.code, "Expected 200 response from #{uri}"
end