ytsync refactor #1 #44

New issue

Closed

opened 2018-09-26 23:13:12 +02:00 by nikooo777 · 2 comments

nikooo777 commented

2018-09-26 23:13:12 +02:00

(Migrated from github.com)

The codebase needs to be heavily refactored.
This issue wants to track the first stage of refactoring which aims at:

Using factory pattern to reduce errors/mistakes while instantiating the manager or the sync structs
bundling data in structures to avoid passing down long lists of variables
better organizing the code so that it's slightly more readable

The codebase needs to be heavily refactored. This issue wants to track the first stage of refactoring which aims at: - Using factory pattern to reduce errors/mistakes while instantiating the manager or the sync structs - bundling data in structures to avoid passing down long lists of variables - better organizing the code so that it's slightly more readable

nikooo777 commented

2018-10-10 00:29:14 +02:00

(Migrated from github.com)

a first part of the refactor was merged as it's working but wasn't reviewed.

lyoshenka commented

2018-10-12 02:23:23 +02:00

(Migrated from github.com)

why did this get assigned to me? i'm closing it. reopen if there's more refactoring to do

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry.go#44

No description provided.

Rows
Columns