More flexibility with Addin

[lorenzwalthert]

• let the user specify the transformers, not just the style argument for flexibility. I.e. scope cannot be specified with the style argument alone, because it takes a function name like styler::tidyverse_style (this holds for the Addins only). When the transformers argument is used, this is possible:

style_text("1+1", transformers = tidyverse_style(scope = 'line_break')

• Rename option styler.addins.style -> styler.addins_style_transformer.
• Overhaul documentation for Addins, add life cycle.
• Add manual test instructions for Addins.

[codecov-io]

Codecov Report

Merging #500 into master will decrease coverage by 0.28%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #500      +/-   ##
==========================================
- Coverage   90.26%   89.98%   -0.29%     
==========================================
  Files          39       39              
  Lines        1675     1677       +2     
==========================================
- Hits         1512     1509       -3     
- Misses        163      168       +5

Impacted Files	Coverage Δ
R/style-guides.R	100% <ø> (ø)	⬆️
R/addins.R	0% <0%> (ø)	⬆️
R/transform-files.R	100% <0%> (ø)	⬆️
R/rules-line-break.R	100% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37381a3...941506b. Read the comment docs.

[lorenzwalthert]

@riccardoporreca are you interested in reviewing this? Note that the R option name was changed from styler.addins.style to styler.addins_style_transformer.

[riccardoporreca]

@lorenzwalthert , happy to review this if it can wait until next week, I will not be much on-line before then.

[lorenzwalthert]

Ok. I just need to submit a new version to CRAN by Mai 10, so maybe I ask @jonmcalder to review? Code review could also be done later, it's more about making sure the changes introduced (Addins + R options) are working and I did not make a capital mistake and the documentation is understandable. Different user scenarios described under tests/manualtest, which I tried on macOs.

[riccardoporreca]

@lorenzwalthert, sure, feel free to ping me if I can still do something after May 5 before the CRAN submission, or at a later stage for a more involved code review.

[jonmcalder]

Looks good to me @lorenzwalthert!

[lorenzwalthert]

Thanks guys.