added option to get decimal point in uncertainty formatted with "parenthesis" notation

[alusiani]

Hi,

I propose a small unobstructive addition to the code in order to optionally get the decimal point formatted when using the "parenthesis" notation. Example of usage:

format(set_errors(123.456, 12.345), notation = "parenthesis", digits=4)
## -> '123.46(1234)'
## default formatting

saved_options = options(errors.parenthesis.unc.dec.point = TRUE)
format(set_errors(123.456, 12.345), notation = "parenthesis", digits=4)
options(saved_options)
## -> '123.46(12.34)'
## optional formatting

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.60%. Comparing base (cc2f192) to head (5b7c42e).
Report is 4 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
- Coverage   97.28%   96.60%   -0.69%     
==========================================
  Files          12       12              
  Lines         516      530      +14     
==========================================
+ Hits          502      512      +10     
- Misses         14       18       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Enchufa2]

Great, thanks for this. I see that some of the changes that I did in the last commits have been undone for some reason. Please restore them. Comments below.

[alusiani]

Sorry for the changes, I should have checked more carefully. I was merging from an older revision I did. Part of the changes were related to using meld with omission of whitespace diffs.

I have now applied all requested changes, committed and pushed to github. As far as I understand, that should be enough, please let me know if I also need to resubmit the pull request.

[Enchufa2]

Thanks! I renamed (and shortened) the new option, and added a new argument decimals to give it more visibility. I also refactored the code a little bit. Since you didn't give me push permission, I'm closing this one in favour of #61, I hope you don't mind.

[alusiani]

Thanks! I renamed (and shortened) the new option, and added a new argument decimals to give it more visibility. I also refactored the code a little bit. Since you didn't give me push permission, I'm closing this one in favour of #61, I hope you don't mind.

No problem! I am not an expert of pull requests and have not realized that a push permission was asked or required.

[Enchufa2]

It is not required, but it is convenient when, like in this case, the owner of the repo wants to add some adjustments to the PR.