Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#26, #27 - Implement Strategy Constraints and FlexibleRollout Strategy #34

Merged
merged 1 commit into from Aug 4, 2020

Conversation

rarruda
Copy link
Collaborator

@rarruda rarruda commented Jul 28, 2020

I should consider adding more tests in the flexible_rollout_spec.rb

@rarruda rarruda self-assigned this Jul 28, 2020
@coveralls
Copy link

Pull Request Test Coverage Report for Build 238

  • 67 of 67 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.08%) to 99.251%

Totals Coverage Status
Change from base Build 235: 0.08%
Covered Lines: 663
Relevant Lines: 668

💛 - Coveralls

@coveralls
Copy link

coveralls commented Jul 28, 2020

Pull Request Test Coverage Report for Build 244

  • 72 of 72 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.09%) to 99.257%

Totals Coverage Status
Change from base Build 235: 0.09%
Covered Lines: 668
Relevant Lines: 673

💛 - Coveralls

@ivarconr
Copy link
Member

Cool ;)

@rarruda rarruda force-pushed the feature/strategy_constraints branch from cb5c193 to 9902be6 Compare August 4, 2020 20:32
@rarruda rarruda force-pushed the feature/strategy_constraints branch from 9902be6 to fe71a29 Compare August 4, 2020 20:33
@sonarcloud
Copy link

sonarcloud bot commented Aug 4, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@rarruda rarruda merged commit 491420f into master Aug 4, 2020
@rarruda rarruda deleted the feature/strategy_constraints branch August 4, 2020 20:48
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.

None yet

3 participants