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

R4R: Add length caps for governance proposal titles and descriptions #3434

Merged
merged 3 commits into from
Jan 29, 2019

Conversation

sunnya97
Copy link
Member

@sunnya97 sunnya97 commented Jan 29, 2019

closes #3420

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

  • Wrote tests

  • Updated relevant documentation (docs/)

  • Added entries in PENDING.md with issue #

  • rereviewed Files changed in the github PR explorer


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@sunnya97 sunnya97 changed the title WIP: Add length caps for governance proposal titles and descriptions R4R: Add length caps for governance proposal titles and descriptions Jan 29, 2019
@sunnya97 sunnya97 added C:x/gov T: API Breaking Breaking changes that impact APIs and the SDK only (not state machine). labels Jan 29, 2019
x/gov/msgs.go Outdated Show resolved Hide resolved
x/gov/msgs.go Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Jan 29, 2019

Codecov Report

Merging #3434 into develop will decrease coverage by 0.07%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #3434      +/-   ##
===========================================
- Coverage    59.36%   59.28%   -0.08%     
===========================================
  Files          134      134              
  Lines         9972     9953      -19     
===========================================
- Hits          5920     5901      -19     
  Misses        3679     3679              
  Partials       373      373

@jackzampolin jackzampolin merged commit 172a451 into develop Jan 29, 2019
@jackzampolin jackzampolin deleted the sunny/gov-description-cap branch January 29, 2019 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/gov T: API Breaking Breaking changes that impact APIs and the SDK only (not state machine).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a simple maximum length for validator descriptions
4 participants