tangled.org / core
Monorepo for Tangled — https://tangled.org
fork atom

appview/notify: pass logger with mergedLogger #693

merged
opened by boltless.me targeting master from boltless.me/core: feat/search
STACK 13
#711 appview/notify: merge NewPullClosed/Merged/Reopen to NewPullState
1
#6
#710 appview: add pull_reopen event
0
#6
#709 appview/pulls: add search endpoint
0
#6
#708 appview/indexer: add pulls indexer
0
#6
#707 appview: add issue_reopen event
0
#6
#706 appview/indexer: put bleve index dir under indexes/ dir
0
#6
#693 appview/notify: pass logger with mergedLogger
0
#9
#695 appview/indexer: handle NewIssueClosed
0
#9
#694 appview/notify: add DeleteIssue event
0
#9
#496 appview: add issue search endpoint
0
#13
#495 wip: add indexer mappings
0
#14
#494 appview: add basic issue indexer (wip)
0
#14
#685 go.mod: add bleve as a dependency
0
#10
0
by boltless.me 0 comments
expand 1 commit
c9316668
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
1
by boltless.me 2 comments
expand 1 commit
debfd60a
appview/notify: pass logger with mergedLogger
oppi.li

nice lgtm, i was considering add a "logger" notifier, separate from every other notifier.

boltless.me

Each notifiers will need their own separate logs, so I think it would be better to let them call logger individually. For example, in same NewIssue event, one notifier might need to log the failure while all other notifiers worked fine.

sign up or login to add to the discussion
2
by boltless.me 0 comments
expand 1 commit
7a6d7b11
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
3
by boltless.me 0 comments
expand 1 commit
079494ce
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
4
by boltless.me 0 comments
expand 1 commit
079494ce
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
5
by boltless.me 0 comments
expand 1 commit
7f2340cf
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
6
by boltless.me 0 comments
expand 1 commit
53d11e27
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
7
by boltless.me 0 comments
expand 1 commit
a89f3ff2
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
8
by boltless.me 0 comments
expand 1 commit
575a9958
appview/notify: pass logger with mergedLogger
sign up or login to add to the discussion
9
by boltless.me 0 comments
expand 1 commit
21e8d9ec
appview/notify: pass logger with mergedLogger
pull request successfully merged
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3m3q47rjacq22