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

Dev 1028 #1088

Merged
merged 2 commits into from
Aug 9, 2023
Merged

Dev 1028 #1088

merged 2 commits into from
Aug 9, 2023

Commits on Aug 9, 2023

  1. 942 install sass if missing (#1028)

    * refactor: add helper for installation of a single dev dep
    
    - choose a good installer for "sass" following the style of install_dev_deps()
    - fix minor typo inside doc of install_dev_deps()
    
    * fix: fix the bug at "sass::" line of dev/run_dev.R
    
    - check if dev/run_dev.R has a "sass"-line a-la "sass::"
    - use the installer install_single_dev_dep() to install SASS
    
    The user is usually asked as force_install=FALSE. Can be changed internally later so pak::pak or install.packages is used in interactive mode (or the behaviour from install_dev_deps if non-interactive).
    ilyaZar authored Aug 9, 2023
    Configuration menu
    Copy the full SHA
    6c5b5f6 View commit details
    Browse the repository at this point in the history
  2. feat

    add httpuv in dev_deps
    add dev_deps parameters in install_dev_deps to allow specific deps installation
    use attachment::att_from_rscript to detect run_dev.R dependencies
    VincentGuyader committed Aug 9, 2023
    Configuration menu
    Copy the full SHA
    933dec5 View commit details
    Browse the repository at this point in the history