Skip to content
This repository was archived by the owner on Dec 10, 2020. It is now read-only.

Commit bdd4faa

Browse files
committed
Lint lib/sync
1 parent 805a87c commit bdd4faa

File tree

5 files changed

+14
-8
lines changed

5 files changed

+14
-8
lines changed

lib/sync/fastsync.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ export class FastSynchronizer extends Synchronizer {
132132
* @param {Peer} peer peer
133133
* @return {Promise}
134134
*/
135-
async announced(announcements: any[], peer: Peer) {
135+
async announced(announcements: any[], _peer: Peer) {
136136
if (announcements.length) {
137137
const [hash, height] = announcements[announcements.length - 1]
138138
this.logger.debug(`New height: number=${height.toString(10)} hash=${short(hash)}`)

lib/sync/fetcher/blockfetcher.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,8 @@ export class BlockFetcher extends Fetcher {
9999
* @param job job
100100
* @return {Peer}
101101
*/
102-
peer(job: any): Peer {
102+
// TODO: find out what _job is supposed to be doing here...
103+
peer(_job: any): Peer {
103104
return this.pool.idle((p: any) => p.eth)
104105
}
105106
}

lib/sync/fetcher/fetcher.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,8 @@ export class Fetcher extends Readable {
124124
if (job.state !== 'active') return
125125
if (result === undefined) {
126126
this.enqueue(job)
127+
// TODO: should this promise actually float?
128+
// eslint-disable-next-line @typescript-eslint/no-floating-promises
127129
this.wait().then(() => {
128130
job.peer.idle = true
129131
})
@@ -270,7 +272,8 @@ export class Fetcher extends Readable {
270272
* @param job job
271273
* @return {Peer}
272274
*/
273-
peer(job?: any) {
275+
// TODO: what is job supposed to be?
276+
peer(_job?: any) {
274277
return this.pool.idle()
275278
}
276279

@@ -280,7 +283,7 @@ export class Fetcher extends Readable {
280283
* @param peer
281284
* @return {Promise}
282285
*/
283-
request(job?: any, peer?: any): Promise<any> {
286+
request(_job?: any, _peer?: any): Promise<any> {
284287
throw new Error('Unimplemented')
285288
}
286289

@@ -290,7 +293,7 @@ export class Fetcher extends Readable {
290293
* @param {Peer} peer peer that handled task
291294
* @param result result data
292295
*/
293-
process(job?: any, peer?: any, result?: any) {
296+
process(_job?: any, _peer?: any, _result?: any) {
294297
throw new Error('Unimplemented')
295298
}
296299

@@ -316,7 +319,7 @@ export class Fetcher extends Readable {
316319
* @param result fetch result
317320
* @return {Promise}
318321
*/
319-
async store(result?: any) {
322+
async store(_result?: any) {
320323
throw new Error('Unimplemented')
321324
}
322325

lib/sync/fetcher/headerfetcher.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@ export class HeaderFetcher extends BlockFetcher {
7272
* @param job job
7373
* @return {Peer}
7474
*/
75-
peer(job: any): Peer {
75+
// TODO: what is job supposed to be?
76+
peer(_job: any): Peer {
7677
return this.pool.idle((p: any) => p.les && p.les.status.serveHeaders)
7778
}
7879
}

lib/sync/sync.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,8 @@ export class Synchronizer extends EventEmitter {
7676
* Returns true if peer can be used for syncing
7777
* @return {boolean}
7878
*/
79-
syncable(peer: any): boolean {
79+
// TODO: evaluate syncability of peer
80+
syncable(_peer: any): boolean {
8081
return true
8182
}
8283

0 commit comments

Comments
 (0)