Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add prefix and new Addins #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

statnmap
Copy link

@statnmap statnmap commented Apr 5, 2021

  • I think that renaming the current Rmd is still useful in the addins list. So I propose to add it back but in a separate addin from the "select File" one.
  • I added the possibility to define myself the prefix of the chunk name as my very long filenames take too much space in my chunk names. The prefix proposed is pass through the clean_latex_special_characters() to be sure everything is good.
  • I updated the unit test accordingly
  • As I was in the Addins files, I added the "choose a directory" feature.

The 3 addins:
image

The interface to choose for "prefix" when using Addin for the "Current Rmd"
image

- Name current Rmd file
- Name file chosen in a directory
- Name chunks from a folder
- issue jumpingrivers#13

- Allow to use 'prefix' in the Addin for current file in relation to
issue jumpingrivers#2
@coveralls
Copy link

Pull Request Test Coverage Report for Build 95

  • 5 of 11 (45.45%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-4.6%) to 90.164%

Changes Missing Coverage Covered Lines Changed/Added Lines %
R/name_chunks_addin.R 0 6 0.0%
Totals Coverage Status
Change from base Build 94: -4.6%
Covered Lines: 110
Relevant Lines: 122

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants