acidhelm / challonge_mgr
85%
master: 96%

LAST BUILD BRANCH: dependabot/bundler/puma-3.12.4
DEFAULT BRANCH: master
Repo Added
Files 22
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH move-calls-to-models
branch: move-calls-to-models
CHANGE BRANCH
x

84

push

travis-ci-com

acidhelm 
Simplify get_api_url call sites
All API URLs start with "tournaments", so don't require the caller to pass
that component in its string.

551 of 647 relevant lines covered (85.16%)

13.15 hits per line

Relevant lines Covered
647 RELEVANT LINES 551 COVERED LINES
13.15 HITS PER LINE
Source Files on move-calls-to-models
Coverage ∆ File Lines Relevant Covered Missed Hits/Line

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
84 move-calls-to-models Simplify get_api_url call sites All API URLs start with "tournaments", so don't require the caller to pass that component in its string. push acidhelm travis-ci-com pending completion  
83 move-calls-to-models Move some of the code that calls Challonge APIs to ChallongeHelper Continuing the effort to make the code organized better, I created `ChallongeHelper`. Classes that call Challonge can now do `include ChallongeHelper` and call its methods directl... push acidhelm travis-ci-com pending completion  
82 move-calls-to-models Add Match#update_scores This replaces direct calls to `ApplicationHelper.update_match`. And I was even able to hide the complexity of making the scores_csv string to pass to the API. That let me make `Match#make_scores_csv` protected. push acidhelm travis-ci-com pending completion  
See All Builds (186)