-
Notifications
You must be signed in to change notification settings - Fork 1
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
Release 1.1.0 #353
Release 1.1.0 #353
Conversation
…from-whole-internal-platforms 314 bug when pulling repos from whole internal platforms
…, refactor GitHub files query, add new info to files table and add new column (default_branch) to repositories output table. In GitHub there is need to define `default_branch` ex ante, so there is need for a repository iterator over `getFilesFromRepo` - this is not the case for GitLab, where GraphQL Query finds itself a default branch.
…nction 200 add get files function
… out and kill all the process if e.g. graphql can not find a user with the login that does not match git org.
Fix pulling users when error on response occurs.
… importantly, a message handler has been added to tackle 403 error on public GitHub side. Also, a new `silence` parameter has been added, as a new function is simply a wrapper over existing ones, and calling it with all the messages would be superfluous. Some optimization in expect-tests.
…y errors), fixing search responses, changing logic of checking usage of a package (two separate functions); also as a side effect of previous attempts add possibility to pull files on given repos (although it is not used now, it may be in the future). Additionaly remove `silence` field from settings as there is no need to use it, change name of check_package to check_R_package.
…me columns of repository output to standardize with files output. Add pulling files by repositories option to GitLab which was missing.
…e-function 326 new search package usage function
…de-specific-list-of-orgrepos-rather-than-pull-entire-org Add possibility to pass vector of repos to host instead of whole orgs
…-files' of https://github.com/r-world-devs/gitstats into 341-when-running-multiple-pull_r_package_use-use-pulled-files
…ctor check_package_as_dependency. Refactoring of the function is swithcing totally to search rest api with filtering response by files (NAMESPACE, DESCRIPTION) instead of pulling them in separate process which was very time consuming. Fixing pulling files for setup repositories (i.e. when they are used instead of pulling new) is a side-effect of earlier tries to optimize the check_package_as_dependency function. This feature is not now used in the function, but it may be helpful in future developments.
…ull_r_package_use-use-pulled-files 341 fix and speed up pulling package usage
…or repositories and commits.
349 prepare 1.1.0 release
Release 1.1.0
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.
Everything is fine based on previous reviews! Congrats on new release! <3
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #353 +/- ##
===========================================
+ Coverage 74.34% 84.59% +10.25%
===========================================
Files 11 15 +4
Lines 998 1948 +950
===========================================
+ Hits 742 1648 +906
- Misses 256 300 +44 ☔ View full report in Codecov by Sentry. |
For the changes see
NEWS.md