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

Add a hash of hash response to Heartcheck #33

Merged
merged 10 commits into from
Apr 12, 2021
Merged

Add a hash of hash response to Heartcheck #33

merged 10 commits into from
Apr 12, 2021

Conversation

LuccaRosa
Copy link

This feature allows the user to choose between a JSON response with hashes { "Some Check": { "status": "ok", "time": 123.45}} and with an array [{"Some check": { "status": "ok"}, "time": 123.45}] using the boolean argument hash_response at the setup.

@sourcelevel-bot
Copy link

Hello, @LuccaRosa! This is your first Pull Request that will be reviewed by SourceLevel, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

@LuccaRosa LuccaRosa requested a review from rhruiz May 18, 2020 16:15
Add a hash of hash response to Heartcheck
Copy link
Contributor

@fabioperrella fabioperrella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LuccaRosa I think it is missing to comment about this new configuration on README and mention this change on CHANGELOG

lib/heartcheck/executors/hash_response.rb Outdated Show resolved Hide resolved
lib/heartcheck/executors/hash_response.rb Outdated Show resolved Hide resolved
lib/heartcheck/executors/hash_response.rb Outdated Show resolved Hide resolved
spec/lib/heartcheck/executors/hash_response_spec.rb Outdated Show resolved Hide resolved
spec/lib/heartcheck/executors/hash_response_spec.rb Outdated Show resolved Hide resolved
spec/lib/heartcheck/executors/hash_response_spec.rb Outdated Show resolved Hide resolved
spec/lib/heartcheck/executors/hash_response_spec.rb Outdated Show resolved Hide resolved
lib/heartcheck.rb Outdated Show resolved Hide resolved
@LuccaRosa LuccaRosa force-pushed the add-hash-return branch 2 times, most recently from 89f4555 to b8dbb74 Compare May 26, 2020 15:12
lib/heartcheck.rb Outdated Show resolved Hide resolved
lib/heartcheck/controllers/base.rb Outdated Show resolved Hide resolved
lib/heartcheck.rb Outdated Show resolved Hide resolved
lib/heartcheck.rb Outdated Show resolved Hide resolved
lib/heartcheck.rb Outdated Show resolved Hide resolved
@angeliski
Copy link
Contributor

Salve galera, podemos mergear esse PR? Acredito que ele vai possibilitar algumas customizações legais que a gente pode querer fazer aqui na empresa :)

@sourcelevel-bot
Copy link

SourceLevel has finished reviewing this Pull Request and has found:

  • 1 fixed issue! 🎉

See more details about this review.

@lorn lorn merged commit 62f72d3 into master Apr 12, 2021
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.

5 participants