Need an "Upload All Dependencies" button #600

[Eduardodudu]

Task

Feature task on the option to upload all packages #600

This PR adds:

1. A structure for uploading all packages under package dependencies
2. It only updates packages that are not already uploaded
3. Reuses the component of uploading single packages
4. Added some lint for the code

[codecov]

Codecov Report

Merging #616 (ef905bc) into dev (fa41c3a) will decrease coverage by 0.66%.
The diff coverage is 8.75%.

❗ Current head ef905bc differs from pull request most recent head 419e57c. Consider uploading reports for the commit 419e57c to get more accurate results

@@            Coverage Diff             @@
##              dev     #616      +/-   ##
==========================================
- Coverage   77.32%   76.66%   -0.66%     
==========================================
  Files          29       29              
  Lines        3863     3904      +41     
==========================================
+ Hits         2987     2993       +6     
- Misses        876      911      +35     

Files Changed	Coverage Δ
R/mod_packageDependencies.R	15.85% <8.75%> (-2.43%)	⬇️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[Robert-Krajcik]

Hi @Eduardodudu

Thanks for working on this.
Since I wrote the mod-packageDepencencies code, I wanted to take a look at your PR.
I just have one comment:
Base R packages should not be included in the count of packages to be loaded, nor should they be loaded.
The message here should be
"There are no new packages to upload."

Thanks!

Suggested change at about line 336:
pkgs_update <- pkg_df() %>%
dplyr::filter(is.na(score) | score == "") %>%
dplyr::filter(!name %in% c(rownames(installed.packages(priority = "base"))))

[Eduardodudu]

Hi @Eduardodudu

Thanks for working on this. Since I wrote the mod-packageDepencencies code, I wanted to take a look at your PR. I just have one comment: Base R packages should not be included in the count of packages to be loaded, nor should they be loaded. The message here should be "There are no new packages to upload."

Thanks!

Suggested change at about line 336: pkgs_update <- pkg_df() %>% dplyr::filter(is.na(score) | score == "") %>% dplyr::filter(!name %in% c(rownames(installed.packages(priority = "base"))))

Thank you @Robert-Krajcik !! Updated!!

[Robert-Krajcik]

This looks really good, Eduardo.
Just one more issue.
Can you get rid of this message?}
The name provided ('fas fa-upload fa-solid') does not correspond to a known icon

[Robert-Krajcik]

Looks good to me!