xml-to-react icon indicating copy to clipboard operation
xml-to-react copied to clipboard

Added optional debug flag to output helpful information to the console

Open wintondeshong opened this issue 5 years ago • 1 comments

Description

Added opt-in flag for constructor so developers can easily troubleshoot what the library is doing.

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] Chore, documentation, cleanup

Related Issue

Motivation and Context

An immediate need is for configuring converters. Instead of it silently skipping a node/children when a tagName doesn't map. After adding those log statements development was much quicker.

How Has This Been Tested?

Additional jest unit tests as well as tested in personal project.

Screenshots (if appropriate):

Checklist:

  • [x] My code follows the code style of this project.
  • [x] I have updated the documentation (if required).
  • [x] I have read the CONTRIBUTING document.
  • [x] I have added tests to cover my changes.
  • [x] All new and existing tests passed.

wintondeshong avatar Jan 23 '20 16:01 wintondeshong

Pull Request Test Coverage Report for Build 459

  • 4 of 4 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 444: 0.0%
Covered Lines: 63
Relevant Lines: 63

💛 - Coveralls

coveralls avatar Jan 23 '20 17:01 coveralls