#332 rounding down of passed percentage

[ed-snodgrass]

🤔 What's changed?

The code was modified to round down rather than round up and to allow 1 decimal place

⚡️ What's your motivation?

Fixes #332

🏷️ What kind of change is this?

• 🐛 Bug fix (non-breaking change which fixes a defect)

♻️ Anything particular you want feedback on?

I would like to know if folks think it should still only display round numbers and/or if it should only round down when between 99% and 100%

📋 Checklist:

• I agree to respect and uphold the Cucumber Community Code of Conduct
• I've changed the behaviour of the code
  • I have added/updated tests to cover my changes.
• My change requires a change to the documentation.
  • I have updated the documentation accordingly.
• Users should know about my change
  • I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

---

This text was originally generated from a template, then edited by hand. You can modify the template here.

[netlify]

✅ Deploy Preview for cucumber-react-preview ready!

Name	Link
🔨 Latest commit	871a9c8
🔍 Latest deploy log	https://app.netlify.com/sites/cucumber-react-preview/deploys/680a59799adea70008f38f34
😎 Deploy Preview	https://deploy-preview-380--cucumber-react-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[davidjgoss]

Thanks for putting this together @ed-snodgrass. To your questions:

I would like to know if folks think it should still only display round numbers

I think a single decimal place would be a good balance if you're up for changing that too.

and/or if it should only round down when between 99% and 100%

I think, especially when combined with 1 dp per above, always flooring will be fine.

[ed-snodgrass]

I agree with one dp. I initially had it that way and then went back to round numbers just in case. I'll move that back to one.