Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Sep 20, 2017

Todo:

  • Change Clients.All to push out selectively as the BuildStatus hub does
  • Allow the build config to pass all the way through registration - currently hardcoded as Ace Commit
  • Set up nightly build which collects coverage (adding it to commit doubles build time)
  • Take specific buildid out of url and revert to latest
  • Add comments around the design of the BuildStatistics directive or rewrite
    • The directive cannot work with a collection if that collection is replaced by the hub
    • The current design binds to a container then dips into the collection when it needs to
  • Consolidate code shared between BuildService and BuildStatisticsService
  • Ensure errors in getting a response are handled
  • Ensure no coverage returned is handled (may not be necessary)

Preview:

image

@ghost ghost requested a review from AButler September 20, 2017 21:09
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.

0 participants