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

feat: fix update public plan proposal & disable codecov patch #133

Merged

Conversation

jaybxyz
Copy link
Contributor

@jaybxyz jaybxyz commented Sep 23, 2021

Description

closes: #131

Tasks

  • Change UpdatePublicPlanProposal logic to update plan type when there is a request
  • Add test codes to cover the updated cases
  • disable codecov patch (reference the comments below)

References

codecov/patch is unreliable and presents weird behaviour from time to time


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Appropriate labels applied
  • Targeted PR against correct branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Re-reviewed Files changed in the Github PR explorer
  • Review Codecov Report in the comment section below once CI passes

@jaybxyz jaybxyz self-assigned this Sep 23, 2021
@codecov
Copy link

codecov bot commented Sep 23, 2021

Codecov Report

Merging #133 (67d3397) into master (85846ba) will increase coverage by 0.71%.
The diff coverage is 51.72%.

❗ Current head 67d3397 differs from pull request most recent head e9381d3. Consider uploading reports for the commit e9381d3 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #133      +/-   ##
==========================================
+ Coverage   69.71%   70.42%   +0.71%     
==========================================
  Files          25       25              
  Lines        2262     2303      +41     
==========================================
+ Hits         1577     1622      +45     
+ Misses        533      518      -15     
- Partials      152      163      +11     
Impacted Files Coverage Δ
x/farming/types/proposal.go 2.80% <0.00%> (-0.82%) ⬇️
x/farming/keeper/proposal_handler.go 62.66% <88.23%> (+25.82%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 85846ba...e9381d3. Read the comment docs.

…131-fix-update-public-proposal

* commit '85846bac8ea958afc14475ee32b7a5f9ae501bb8':
  docs: update issue, pr template
  docs: add bedges and update issue template
  test: add more test cases for edge case
  test: add test case for tendermint#134
  chore: fix typo
  fix: use IsAllLTE instead of IsAllLT
@jaybxyz jaybxyz marked this pull request as ready for review September 24, 2021 05:43
@jaybxyz
Copy link
Contributor Author

jaybxyz commented Sep 24, 2021

Copy link
Contributor

@dongsam dongsam left a comment

Choose a reason for hiding this comment

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

LGTM!

…131-fix-update-public-proposal

* commit '9c24552e9d7e3609e99f6803d3b98f6539267613':
  fix: broken link
  chore: fix broken links and add CONTRIBUTING.md
  feat: add markdown link checker workflow
@jaybxyz jaybxyz changed the title feat: fix update public plan proposal feat: fix update public plan proposal & disable codecov patch Sep 27, 2021
@jaybxyz jaybxyz merged commit 97b297b into tendermint:master Sep 27, 2021
jaybxyz added a commit to jaybxyz/farming that referenced this pull request Sep 27, 2021
…121-add-demo-docs

* commit '97b297b45a792b0ffc6616e24595c314aecf89b6':
  feat: fix update public plan proposal & disable codecov patch (tendermint#133)
  fix: broken link
  chore: fix broken links and add CONTRIBUTING.md
  feat: add markdown link checker workflow
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.

Update public proposal to change plan type doesn't work
3 participants