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.

Sign up for GitHub

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

Jump to bottom

Remove console.log in classDB. #861

Merged
merged 1 commit into from
Jun 25, 2019
Merged

Remove console.log in classDB. #861

merged 1 commit into from
Jun 25, 2019

Conversation

Arthaey
Copy link
Contributor

@Arthaey Arthaey commented Jun 24, 2019

No description provided.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 784

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.008%) to 52.904%
Totals Coverage Status
Change from base Build 783: -0.008%
Covered Lines: 2276
Relevant Lines: 4241
💛 - Coveralls

@Arthaey
Copy link
Contributor Author

Arthaey commented Jun 24, 2019

The entire diff of this PR is to remove a console.log from a method. While technically that decreased code coverage by -0.008%, hopefully you'll let that technicality slide in this case. ;)

@knsv knsv merged commit f2137e4 into mermaid-js:master Jun 25, 2019
@knsv
Copy link
Collaborator

knsv commented Jun 25, 2019

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Arthaey @coveralls @knsv