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

changelog: Reorder items in the changelog #1611

Merged
merged 2 commits into from
Jul 10, 2018
Merged

Conversation

ValarDragon
Copy link
Contributor

@ValarDragon ValarDragon commented Jul 10, 2018

I've reordered the entries to have entries tagged under the same module put together. (I think it was kinda weird with them not together before)

Additionally I've added the following entries:

  • Added MinInt and MinUint functions

    • The keys sub-module is now in the sdk
  • [keys] New keys now have 24 word recovery keys, for heightened security

  • Key Generation Errors #872 - recovery phrases no longer all end in abandon

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • [ ]~ Wrote tests~
  • Updated CHANGELOG.md
  • Updated Gaia/Examples
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)

Feel free to close this issue though, if there was some ordering, or we think that any sort of ordering doesn't matter. I don't feel strongly about it

Additionally I've added the following entries:

* Added MinInt and MinUint functions

     * The keys sub-module is now in the sdk

* [keys] New keys now have 24 word recovery keys, for heightened security

* \#872  - recovery phrases no longer all end in `abandon`
@codecov
Copy link

codecov bot commented Jul 10, 2018

Codecov Report

❗ No coverage uploaded for pull request base (develop@9531466). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #1611   +/-   ##
==========================================
  Coverage           ?   64.12%           
==========================================
  Files              ?      122           
  Lines              ?     6754           
  Branches           ?        0           
==========================================
  Hits               ?     4331           
  Misses             ?     2177           
  Partials           ?      246

@ebuchman ebuchman merged commit 4ef83bf into develop Jul 10, 2018
@ebuchman ebuchman deleted the dev/update_changelog branch July 10, 2018 04:18
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.

2 participants