Skip to content

Remove panic when method not found in the tree#43

Merged
xujiajun merged 8 commits intoxujiajun:developfrom
tadvi:master
Mar 9, 2019
Merged

Remove panic when method not found in the tree#43
xujiajun merged 8 commits intoxujiajun:developfrom
tadvi:master

Conversation

@tadvi
Copy link
Contributor

@tadvi tadvi commented Mar 7, 2019

Replaced panic with HandleNotFound in router.go line 257.

@coveralls
Copy link

coveralls commented Mar 7, 2019

Pull Request Test Coverage Report for Build 190

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.7%) to 99.324%

Files with Coverage Reduction New Missed Lines %
router.go 2 99.05%
Totals Coverage Status
Change from base Build 183: -0.7%
Covered Lines: 294
Relevant Lines: 296

💛 - Coveralls

@xujiajun xujiajun merged commit 03e0408 into xujiajun:develop Mar 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants