Skip to content
This repository was archived by the owner on Apr 3, 2026. It is now read-only.

WIP, refactoring all library#304

Open
netzulo wants to merge 49 commits into
masterfrom
remove-useless
Open

WIP, refactoring all library#304
netzulo wants to merge 49 commits into
masterfrom
remove-useless

Conversation

@netzulo

@netzulo netzulo commented Jan 29, 2020

Copy link
Copy Markdown
Owner

@netzulo netzulo added Feature New functionality proposal Best Practice Not functionality problem, but refactor need it Testcase Documentation Related to documentation CI Related to some Continuous Integration labels Jan 29, 2020
@netzulo netzulo self-assigned this Jan 29, 2020
@netzulo netzulo linked an issue Apr 6, 2020 that may be closed by this pull request
@qlty-cloud-legacy

qlty-cloud-legacy Bot commented May 25, 2021

Copy link
Copy Markdown

Code Climate has analyzed commit fa460d6 and detected 4 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 3
Clarity 1

View more on Code Climate.

@qlty-cloud-legacy qlty-cloud-legacy Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 7698 lines exceeds the maximum allowed for the inline comments feature.

@qlty-cloud-legacy qlty-cloud-legacy Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 7698 lines exceeds the maximum allowed for the inline comments feature.

@netzulo netzulo left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok at this point

@qlty-cloud-legacy qlty-cloud-legacy Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 7746 lines exceeds the maximum allowed for the inline comments feature.

@qlty-cloud-legacy qlty-cloud-legacy Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 7371 lines exceeds the maximum allowed for the inline comments feature.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Best Practice Not functionality problem, but refactor need it Bug Problem in software code CI Related to some Continuous Integration dependencies Pull requests that update a dependency file Documentation Related to documentation Feature New functionality proposal Testcase

Projects

None yet

1 participant