Skip to content
This repository has been archived by the owner on Jun 22, 2023. It is now read-only.

Update version. #77

Merged
merged 1 commit into from
Aug 7, 2015
Merged

Update version. #77

merged 1 commit into from
Aug 7, 2015

Conversation

sjstack
Copy link
Contributor

@sjstack sjstack commented Aug 7, 2015

Description:

Review:

  • Do the unit tests contain positive functionality tests as well as edge case negative tests?
  • Can we monitor the performance/effectiveness of these changes?
  • Did the code review consider failure modes and edge cases?
  • Did the code review consider refactorings that would make this code more maintainable?
  • Was this change approved by a module owner (if necessary)?

jeffzhou791 pushed a commit that referenced this pull request Aug 7, 2015
Merge pull request #77 from CloudHealth/update_version_after_adding_e…
8d01d99 
…lasticache_pricing

Update version.
@jeffzhou791 jeffzhou791 merged commit 8d01d99 into master Aug 7, 2015
@sjstack sjstack deleted the update_version_after_adding_elasticache_pricing branch August 7, 2015 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sjstack @jeffzhou791