Skip to content

Commit

Permalink
Merge pull request #306 from r-world-devs/Maciej-305-fix-and-neat-cmd…
Browse files Browse the repository at this point in the history
…-checks

Update checks.
  • Loading branch information
maciekbanas authored Sep 28, 2023
2 parents 18e811e + b0087de commit 082a61a
Show file tree
Hide file tree
Showing 14 changed files with 68 additions and 296 deletions.
227 changes: 0 additions & 227 deletions .github/workflows/code-coverage.yaml

This file was deleted.

16 changes: 0 additions & 16 deletions .github/workflows/other-checks.yaml

This file was deleted.

5 changes: 3 additions & 2 deletions DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,12 @@ Imports:
lubridate,
magrittr,
plotly,
progress,
rlang (>= 1.1.0),
R6,
purrr (>= 1.0.0),
stringr
Suggests:
Suggests:
spelling,
covr,
knitr,
mockery,
Expand All @@ -37,3 +37,4 @@ Suggests:
withr
Config/testthat/edition: 3
VignetteBuilder: knitr
Language: en-US
1 change: 0 additions & 1 deletion NAMESPACE
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ importFrom(lubridate,floor_date)
importFrom(magrittr,"%>%")
importFrom(plotly,ggplotly)
importFrom(plotly,layout)
importFrom(progress,progress_bar)
importFrom(purrr,keep)
importFrom(purrr,map)
importFrom(purrr,map_chr)
Expand Down
2 changes: 1 addition & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ GitStats (development version)
- changed name of `setup` to `set_params` function (I: #294),
- set new name for `set_connection` function: `set_host` as it is more informative (and shorter) (I: #271),
- changed name of a function: `add_team_member` to `set_team_member` (I: #271),
- added setting tokens by default - if a user does have all the PATs set up in environment variables (as e.g. `GITHUB_PAT` or `GITLAB_PAT`), there is no need to pass them as an arugment to `set_connection` (I: #120 PR: #268),
- added setting tokens by default - if a user does have all the PATs set up in environment variables (as e.g. `GITHUB_PAT` or `GITLAB_PAT`), there is no need to pass them as an argument to `set_connection` (I: #120 PR: #268),
- added `pull_users()` function to pull information on users (I: #199 PR: #238),
- added possibility of scanning whole internal git platforms if no `orgs` are passed (I: #258),
- added switching to REST engine in case GraphQL fails with 502 error (I: #225 PRs: #227 (for repos) #261 (for commits))
Expand Down
1 change: 0 additions & 1 deletion R/EngineGraphQL.R
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
#' @importFrom dplyr distinct mutate relocate filter
#' @importFrom progress progress_bar
#'
#' @title A EngineGraphQL class
#' @description A class for methods wrapping GitHub's GraphQL API responses.
Expand Down
1 change: 0 additions & 1 deletion R/EngineGraphQLGitHub.R
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
#' @importFrom dplyr distinct mutate filter
#' @importFrom progress progress_bar
#'
#' @title A EngineGraphQLGitHub class
#' @description A class for methods wrapping GitHub's GraphQL API responses.
Expand Down
2 changes: 1 addition & 1 deletion R/EngineRestGitHub.R
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ EngineRestGitHub <- R6::R6Class("EngineRestGitHub",
NULL
},

#' @description A suportive method to get commits, run when GraphQL fails.
#' @description A supportive method to get commits, run when GraphQL fails.
#' @param org An organization of repositories.
#' @param date_from A starting date to look commits for.
#' @param date_until An end date to look commits for.
Expand Down
2 changes: 1 addition & 1 deletion R/GitHost.R
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ GitHost <- R6::R6Class("GitHost",
#' @description A method to list all repositories for an organization, a
#' team or by a keyword.
#' @param settings A list of `GitStats` settings.
#' @param add_contributors A boolean to decide wether to add contributors
#' @param add_contributors A boolean to decide whether to add contributors
#' column to repositories table.
#' @return A data.frame of repositories.
pull_repos = function(settings, add_contributors = FALSE) {
Expand Down
25 changes: 17 additions & 8 deletions inst/WORDLIST
Original file line number Diff line number Diff line change
@@ -1,25 +1,34 @@
Acknowledgement
CMD
Codecov
EngineRest
EngineGraphQL
EngineGraphQLGitHub
EngineGraphQLGitLab
EngineRest
EngineRestGitHub
EngineRestGitLab
Forys
GQLQueryGitHub
GQLQueryGitLab
GitHost
GitLab
GitLab's
GithubMetrics
GraphQL
GQLQuery
GQLQueryGitHub
Igras
Krystian
OOP
PATs
PRs
Wais
cloneable
codelines
endCursor
httr
lifecycle
magrittr
organisations
orgs
param
plotly
repo
repos
orgs
endCursor
httr

2 changes: 1 addition & 1 deletion man/EngineRestGitHub.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion man/GitHost.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 082a61a

Please sign in to comment.