Commit Graph

216 Commits

Author SHA1 Message Date
Brandon Keepers
b9eb512562 Switch GitHub client tests to typescript 2018-04-22 20:12:16 -05:00
Brandon Keepers
a9e7162d11 Merge remote-tracking branch 'origin/master' into typescript-init
* origin/master:
  correct featured apps link (#490)
  chore: Add icon to title of probot (#483)
  docs: Recommend using `npm run dev` to start with nodemon (#487)
2018-04-09 17:46:48 -05:00
Kautilya Tripathi
72d0d81bf8 chore: Add icon to title of probot (#483) 2018-04-05 13:59:44 -05:00
tcbyrd
522c6dff9f Merge branch 'master' into typescript-init 2018-04-01 20:42:04 -04:00
tcbyrd
5b1dd99ed1 Updating testMatchers
This exposed some test that weren't running
2018-04-01 20:29:57 -04:00
Brandon Keepers
6e60927dba fix: allow initializing the GitHub client without options (#479) 2018-03-22 17:33:38 -05:00
tcbyrd
d474fe3539 TS version of #470 (Refactor GitHub client extensions) 2018-03-22 11:12:41 -04:00
tcbyrd
effb3f5f3c Merge tag 'v6.1.0'
Conflicts:
	docs/development.md
	lib/github.js
	lib/github/pagination.js
	lib/index.js
	package.json
	src/github.ts
	src/index.ts

Coverted all lib/github javascript files to ts
2018-03-22 10:01:05 -04:00
Brandon Keepers
3503b1f611 Add GraphQL support (#472) 2018-03-21 21:33:16 -05:00
Brandon Keepers
9f324c79e3 chore: Minimal changes to get @octokit/webhooks working (#462) 2018-03-21 13:23:51 -04:00
Jason Etcovitch
2fc8fd8f45 feat: Better error messages (#434) 2018-03-21 01:33:58 -04:00
tcbyrd
647711ab5a Cleaning up some code and fixing ts errors
Remove --forceExit

Clean up unused code

assign this.id

sync changes from origin
2018-03-15 23:48:21 -04:00
Alexey Alekhin
ddb5b3d9ba Raise an error if there are several .pem files (#441) 2018-03-08 21:33:46 -05:00
tcbyrd
920d5b67f6 Merging master into typescript-init
Reverts https://github.com/probot/probot/pull/397/files

fix(package): update @octokit/rest to version 15.1.7
Name the Logger export

Make object types optional

Merge branch 'master' into typescript-init

Bump deps for standard and octokit

Bump ts dependency

Made the option optional. 😉
2018-03-08 15:54:37 -05:00
greenkeeper[bot]
0060098fd7 chore(package): Update standard to the latest version 🚀 (#436) 2018-02-28 10:09:40 -05:00
tcbyrd
30af8747c2 Refactor imports and exports and fix tests
Convert the rest of the files in `/src` and disable allowJs

Move everything to named exports

Update bin to use named exports

Make all tests js and use named exports
2018-02-25 20:42:25 -05:00
tcbyrd
aed1a7b95d Merge remote-tracking branch 'origin-auth/master' into typescript-init
Not sure why this isn't being ignored
2018-02-18 11:01:34 -05:00
tcbyrd
a4c8e77c82 Fixup serializer test by converting it to TS 2018-02-17 23:55:56 -05:00
tcbyrd
f2266abb5f Convert all the things!
Most of this is pulled over from @philschatz:
https://github.com/probot/probot/compare/master...philschatz:typescript

Some of the async tests are still failing, but lots of forward progress
to get all the core files converted to TS.
2018-02-17 23:23:17 -05:00
Brandon Keepers
72d6911a79 fix: Re-throw errors from octokit (#418) 2018-02-05 10:28:02 -05:00
greenkeeper[bot]
2f3f434ab8 fix(package): Update bottleneck to the latest version 🚀 (#397) 2018-02-03 15:55:50 -05:00
Gregor
5bf1c315c7 test: adapt for @octokit/rest
Turns out that the `per_page` & `page` params are missing in [routes.json](https://github.com/octokit/rest.js/blob/master/lib/routes.json#L1682-L1687), which means, with the previous implementations, they have been ignored!

The new implementation automatically passes all remaining options as query parameter (for `GET` requests) so it works anyway.

So tl;dr: the new release actually fixed a bug, getting the stats will be more performant now, as it gets 100 results instead of 30
2018-01-29 11:07:17 -08:00
Brandon Keepers
0a4b297757 Allow setting logger name to separate components (#355) 2018-01-29 08:26:33 -06:00
tcbyrd
6763956669 Merge branch 'master' into typescript-init
Merge changes from 5.0

Remove merge markers
2018-01-27 18:24:46 -05:00
Wilhelm Klopp
dcd0e32c3c Add option to disable stats (#392) 2018-01-12 08:20:25 -06:00
Brandon Keepers
a3c91afdf3 Properly handle async errors (#384) 2018-01-12 08:10:10 -06:00
Brandon Keepers
0b527178eb Use smee-client instead of EventSource directly (#389) 2018-01-11 21:39:50 -06:00
Ryan Tsao
36e3b57b41 feature: Add early termination to paginate method (#329) 2018-01-11 13:16:44 -05:00
Olivier Combe
5546e5c4fc working typescript setup (#375) 2017-12-30 23:14:26 -05:00
tcbyrd
cfcc87e525 Run the test against the transpiled file 2017-12-17 19:11:39 -05:00
Jason Etcovitch
7bef6a7bf9 chore: Add serializer tests (#369) 2017-12-13 11:13:46 -05:00
Brandon Keepers
6aa3a2f7b2 feat: Add support for experimental webhook proxy (#359)
Usage:

    $ WEBHOOK_PROXY_URL=http://github-webhook-proxy.herokuapp.com/abcxyz npm start
2017-12-07 12:30:18 -06:00
Jason Etcovitch
92c5c52e66 Implement GHE Support (#331)
* Add GHE_URL host option to GitHubApi

* Absorb github-app repo

* Implement new github-app setup

* Basic tests for GHE support

* Add docs

* Switch GHE_URL for GHE_HOST

* Make the tests pass by moving things

* Fix sentry test
2017-11-30 13:02:58 -05:00
Brandon Keepers
27b7c8d406 refactor: Extract Sentry logging to a built-in plugin (#353) 2017-11-27 20:57:52 -06:00
Brandon Keepers
e35028890a Improve GitHub logging (#351)
* Extract github client extensions into module

* Add logging to extended GitHub client

* Update test to check output

* Inline GitHub App methods

* Update node-github links

* Remove src, which does not show up with LOG_FORMAT=simple anyway

* Fix lint errors
2017-11-27 10:18:58 -06:00
Brandon Keepers
1e2551853e feat: Add request/response logging (#322)
BREAKING CHANGE:
- Default log level is now `INFO` instead of `DEBUG`
2017-11-25 22:16:54 -06:00
Brandon Keepers
c5a0c5956c Add context.log to add event context to each log message (#321)
* Add logger on context.log

* Update robot.log instances in docs

* Add more event context
2017-11-21 12:31:55 -06:00
Jason Etcovitch
9eeac3a57e refactor: switch ejs for hbs 2017-11-21 09:21:39 -05:00
Brandon Keepers
5dde8405c5 chore: remove unnecessary test config (#313) 2017-11-02 20:39:12 -05:00
Jason Etcovitch
5d29945dd2 feat: /probot homepage (#279)
* Add /probot route with tests

* Add index route, write tests

* Remove unused arg

* Pass tests

* Tweak HTML string and pkg require

* Add static directory

* Design welcome page

* Rename to probot.html

* Implement EJS

* Remove duplicate load

* Extract default homepage into plugin

* Fix issue after merge conflict

* Refactor app loading

* Remove autoloading of plugings

BREAKING CHANGE: `probot run` without any arguments will no longer autoload apps named `probot-*`.

* Move setup

* Look for `apps` key instead of `plugins`

* Update simulate command

* Simplify resolver

* Remove unused routes variable

* Move views to default path

* Call probot.setup to properly initialize default apps

* Simplify package loading

* Specify extension

* Add links to docs/slack

* Remove stats link

* Add helper to jest ignore

* Fix ejs views error

* Move a require to top of file

* Remove unnecessary (and breaking) test
2017-10-17 09:35:58 -04:00
Jason Etcovitch
e7a0c5cd80 chore(tests): Migrate to Jest
* Add jest, remove mocha/expect

* Migrate all tests to Jest

* Make test pass

* Move tests into `/test/`

* Update testing docs

* Fix resolver test
2017-10-14 06:02:34 -07:00
Brandon Keepers
dda5df88d8 Refactor app loading (#284)
BREAKING CHANGE: `probot run` without any arguments will no longer autoload apps named `probot-*`.
2017-10-14 05:25:02 -07:00
Bex Warner
1227ef9d3b removes unnecessary eslint disables 2017-10-12 14:12:39 -07:00
Brandon Keepers
6d8ddaf3f7 feat: Expose public stats about installations (#181) 2017-10-11 21:09:13 -07:00
Brandon Keepers
1c0657bbbc Remove deprecated features 2017-09-05 12:21:57 -05:00
Brandon Keepers
9250c8c555 Set LOG_LEVEL=fatal for all tests 2017-08-29 22:32:01 -05:00
Brandon Keepers
d441aa6f4c Silence server errors in test with default error handler 2017-08-29 22:11:18 -05:00
Brandon Keepers
e93555397e Remove deprecated event argument for callback 2017-08-24 15:57:13 -05:00
Brandon Keepers
dc33e4be6f Remove deprecated robot method from a probot instance 2017-08-24 15:54:59 -05:00
Brandon Keepers
07c9dd46ae Merge pull request #212 from wilhelmklopp/master
Multiple events per robot.on()
2017-08-24 15:49:27 -05:00