Skip to content

fix: intercase leverage fix#1800

Merged
mergify[bot] merged 9 commits intomainfrom
woz/intercase-fix
Feb 9, 2023
Merged

fix: intercase leverage fix#1800
mergify[bot] merged 9 commits intomainfrom
woz/intercase-fix

Conversation

@adamewozniak
Copy link
Copy Markdown
Collaborator

Description

closes: #XXXX


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • included the correct type prefix in the PR title
  • added ! to the type prefix if API or client breaking change
  • added appropriate labels to the PR
  • targeted the correct branch (see PR Targeting)
  • provided a link to the relevant issue or specification
  • added a changelog entry to CHANGELOG.md
  • included comments for documenting Go code
  • updated the relevant documentation or specification
  • reviewed "Files changed" and left comments if necessary
  • confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

  • confirmed the correct type prefix in the PR title
  • confirmed all author checklist items have been addressed
  • reviewed state machine logic
  • reviewed API design and naming
  • reviewed documentation is accurate
  • reviewed tests and test coverage
  • manually tested (if applicable)

@adamewozniak
Copy link
Copy Markdown
Collaborator Author

needs some testing before merging in

@adamewozniak adamewozniak marked this pull request as ready for review February 9, 2023 20:50
@adamewozniak adamewozniak requested review from a team as code owners February 9, 2023 20:50
Copy link
Copy Markdown
Contributor

@zarazan zarazan left a comment

Choose a reason for hiding this comment

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

@adamewozniak adamewozniak added the S:backport/v4.0.x Backport patch to release/v4.0.x label Feb 9, 2023
@adamewozniak adamewozniak requested a review from a team as a code owner February 9, 2023 21:06
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #1800 (15040e5) into main (2104ccf) will decrease coverage by 0.05%.
The diff coverage is 55.55%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1800      +/-   ##
==========================================
- Coverage   57.85%   57.80%   -0.05%     
==========================================
  Files          78       78              
  Lines        8017     8023       +6     
==========================================
  Hits         4638     4638              
- Misses       3043     3047       +4     
- Partials      336      338       +2     
Impacted Files Coverage Δ
x/leverage/keeper/grpc_query.go 64.23% <50.00%> (-1.37%) ⬇️
x/leverage/keeper/oracle.go 62.01% <100.00%> (ø)

@mergify mergify bot merged commit ea76766 into main Feb 9, 2023
@mergify mergify bot deleted the woz/intercase-fix branch February 9, 2023 21:19
mergify bot pushed a commit that referenced this pull request Feb 9, 2023
## Description

closes: #XXXX

---

### Author Checklist

_All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues._

I have...

- [ ] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] added `!` to the type prefix if API or client breaking change
- [ ] added appropriate labels to the PR
- [ ] targeted the correct branch (see [PR Targeting](https://github.com/umee-network/umee/blob/main/CONTRIBUTING.md#pr-targeting))
- [ ] provided a link to the relevant issue or specification
- [ ] added a changelog entry to `CHANGELOG.md`
- [ ] included comments for [documenting Go code](https://blog.golang.org/godoc)
- [ ] updated the relevant documentation or specification
- [ ] reviewed "Files changed" and left comments if necessary
- [ ] confirmed all CI checks have passed

### Reviewers Checklist

_All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items._

I have...

- [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] confirmed all author checklist items have been addressed
- [ ] reviewed state machine logic
- [ ] reviewed API design and naming
- [ ] reviewed documentation is accurate
- [ ] reviewed tests and test coverage
- [ ] manually tested (if applicable)

(cherry picked from commit ea76766)

# Conflicts:
#	CHANGELOG.md
#	x/leverage/keeper/oracle.go
adamewozniak added a commit that referenced this pull request Feb 9, 2023
* fix: intercase leverage fix (#1800)

## Description

closes: #XXXX

---

### Author Checklist

_All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues._

I have...

- [ ] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] added `!` to the type prefix if API or client breaking change
- [ ] added appropriate labels to the PR
- [ ] targeted the correct branch (see [PR Targeting](https://github.com/umee-network/umee/blob/main/CONTRIBUTING.md#pr-targeting))
- [ ] provided a link to the relevant issue or specification
- [ ] added a changelog entry to `CHANGELOG.md`
- [ ] included comments for [documenting Go code](https://blog.golang.org/godoc)
- [ ] updated the relevant documentation or specification
- [ ] reviewed "Files changed" and left comments if necessary
- [ ] confirmed all CI checks have passed

### Reviewers Checklist

_All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items._

I have...

- [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] confirmed all author checklist items have been addressed
- [ ] reviewed state machine logic
- [ ] reviewed API design and naming
- [ ] reviewed documentation is accurate
- [ ] reviewed tests and test coverage
- [ ] manually tested (if applicable)

(cherry picked from commit ea76766)

# Conflicts:
#	CHANGELOG.md
#	x/leverage/keeper/oracle.go

* fix conflicts

* coin.New fix

* Update CHANGELOG.md

---------

Co-authored-by: Adam Wozniak <29418299+adamewozniak@users.noreply.github.com>
Co-authored-by: Adam Moser <63419657+toteki@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A:Automerge S:backport/v4.0.x Backport patch to release/v4.0.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants