search

Antonboom / testifylint

The Golang linter that checks usage of github.com/stretchr/testify.
https://github.com/stretchr/testify
MIT License
101 stars 8 forks source link

refactor generator #121

Closed ccoVeille closed 5 months ago

ccoVeille commented 5 months ago

These commits were part of ~#107~ and #111 that were suggested changes were discarded, but the refactoring made in each PR could be kept as discussed here

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9436791382

Details

Totals Coverage Status
Change from base Build 9431505288: 0.0%
Covered Lines: 2110
Relevant Lines: 2254
💛 - Coveralls
ccoVeille commented 5 months ago

This PR uses commits available in:

ccoVeille commented 5 months ago

@Antonboom do you think we could merge this one, it would help me to refactor for

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9511399411

Details

Totals Coverage Status
Change from base Build 9431505288: 0.0%
Covered Lines: 2110
Relevant Lines: 2254
💛 - Coveralls
ccoVeille commented 5 months ago

I pushed too fast sorry.

I will fix other too

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9513179523

Details

Totals Coverage Status
Change from base Build 9431505288: 0.0%
Covered Lines: 2110
Relevant Lines: 2254
💛 - Coveralls
ccoVeille commented 5 months ago

I resolved all your comments @Antonboom please check and review.

The number of changes is then very limited

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9513220198

Details

Totals Coverage Status
Change from base Build 9431505288: 0.0%
Covered Lines: 2110
Relevant Lines: 2254
💛 - Coveralls
coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9525679838

Details

Totals Coverage Status
Change from base Build 9431505288: 0.0%
Covered Lines: 2110
Relevant Lines: 2254
💛 - Coveralls