I'm submitting a...

[ ] Bug Fix
[ ] Feature
[ x ] Other (Refactoring, Added tests, Documentation, ...)

Checklist

• [ x ] Test your changes
• [ x ] Followed the build steps

Description

Thank you for accepting the recent pull request on making the endVal argument optional by having the method look at the element inner HTML for a value if null or undefined is given as the endVal value. That pull request should cause minor breakage.

This pull request is to modify the CountUp() method by removing the endVal argument and moving endVal as one of the options in the options argument.

This will allow for the following code to work...

// Without options
const countUp = new CountUp('targetId');

// With options
const countUp = new CountUp('targetId', options);

No need to for code like this...

const countUp = new CountUp('targetId', null, options);

Since endVal is now optional, it should be an option, not a required argument for when you want to set some options.

I also updated the tests, the README.md and demo scripts to include these changes to make this pull request as seamless as possible.

All of the updated tests do pass.

This however will cause breaking changes and I would suggest pushing this for a 3.0.0 release.

Does this PR introduce a breaking change?

[ x ] Yes
[ ] No