-
Notifications
You must be signed in to change notification settings - Fork 10
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
Include add_before and add_after in new item functions #91
Conversation
Merge branch 'develop' into eli/auto-add # Conflicts: # R/case.R # R/code.R # R/req.R # man/new_item.Rd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good with the concept, but if you could go back and clean up the pkg
argument, I think this PR will pass CI and be good to merge in. Hopefully that is quick enough!
there are tests failing now that were explicitly adding the files into the validation.yml! haha. at least we know if works!! |
Yep! I'm working on those now. Should be ready by end of today |
Hey @thebioengineer, I had some trouble with the tests. I think my VM must have a weird pdf render because I get a lot of errors on my machine. But they are passing here so it should be good to merge for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thanks!!
@elimillera did pulling branch |
This PR adds the
add_file_to_config
functionality and adds the parametersadd_before
andadd_after
to be passed into thebefore
andafter
parameters ofadd_file_to_config
.I'm not sure what the plans are for merging things in but I can start working off this branch to add 'tidy_select' functionality if I'm on the right track.
Closes #66