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 dependencies #2
Conversation
created: is(Number), | ||
creator: is(String) | ||
}); | ||
export function isChannel(x: any): x is Channel { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where
が type predicate として振る舞わないので泣く泣くこうしている (isInstantMessage
も同様)
@@ -53,7 +53,7 @@ export function updateChannel(key: string, project: (x: Channel) => Channel): Ac | |||
} | |||
|
|||
export function deleteChannel(key: string): Action { | |||
return curryDel(key); | |||
return over(lensProp('channels'), curryDel(key)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここは元の実装がおかしかった疑惑がある.
@@ -78,7 +78,7 @@ export function makeSlackBotDriver(token: string, options?: makeBotDriverOptions | |||
const pong$: O<Timestamp<{}>> = fromEvent(socket, 'pong') | |||
.timestamp(); | |||
|
|||
const pingOutput$ = ping$.mapTo(() => socket.ping(null, {}, true)); | |||
const pingOutput$ = ping$.mapTo(() => socket.ping(null, true, true)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1,4 +1,4 @@ | |||
import * as ms from 'ms'; | |||
import ms = require('ms'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
かなしい
03a44da
to
703eeeb
Compare
@@ -19,7 +22,7 @@ export class EventSource { | |||
} | |||
|
|||
selectByChannelName(name: string): Observable<IncomingMessage<Channel>> { | |||
return this._message$.mergeMap(({event, channels}) => { | |||
return this._message$.filter<BothPS, ChanPS>((ps: BothPS): ps is ChanPS => isChannel(ps.event.channel)).mergeMap(({event, channels}) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
このあたりも今まで動いてたのが謎
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
最 of 高
@types/ms
@types/ws
@types/node
ms
(0.7.3
->2.0.0
)ws
(1.1.1
->3.2.0
)rxjs
(5.0.3
->5.4.3
)ramda
(0.22.1
->0.25.0
)@types/ramda
(0.0.2
->0.24.17
)