Skip to content

Conversation

@rg911
Copy link
Contributor

@rg911 rg911 commented Sep 20, 2019

No description provided.

@rg911 rg911 requested a review from evias September 20, 2019 10:07
@rg911 rg911 closed this Sep 20, 2019
@coveralls
Copy link

coveralls commented Sep 20, 2019

Pull Request Test Coverage Report for Build 669

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 69.096%

Totals Coverage Status
Change from base Build 665: 0.0%
Covered Lines: 6264
Relevant Lines: 8363

💛 - Coveralls

@rg911 rg911 reopened this Sep 20, 2019
@dgarcia360 dgarcia360 self-requested a review September 20, 2019 10:23
@dgarcia360
Copy link
Contributor

dgarcia360 commented Sep 20, 2019

Thanks for completing the changelog 🥇 The PR looks correct, but please coordinate with the REST team to also tag a partial e3 compatible version, or release it as a "beta" update. https://medium.com/@kevinkreuzer/publishing-a-beta-or-alpha-version-to-npm-46035b630dd7

@rg911 rg911 merged commit 181fe86 into symbol:master Sep 20, 2019
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.

3 participants