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

Release 1.1.0 #353

Merged
merged 41 commits into from
Jan 8, 2024
Merged

Release 1.1.0 #353

merged 41 commits into from
Jan 8, 2024

Conversation

maciekbanas
Copy link
Member

@maciekbanas maciekbanas commented Jan 5, 2024

For the changes see NEWS.md

maciekbanas and others added 30 commits October 25, 2023 13:19
…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.
… 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
maciekbanas and others added 11 commits December 19, 2023 10:42
…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
@maciekbanas maciekbanas requested a review from Cotau January 5, 2024 11:50
@maciekbanas maciekbanas self-assigned this Jan 5, 2024
@maciekbanas maciekbanas changed the title Release 0.5.1 Release 1.1.0 Jan 5, 2024
Copy link
Collaborator

@Cotau Cotau left a 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-commenter
Copy link

Codecov Report

Attention: 209 lines in your changes are missing coverage. Please review.

Comparison is base (f0ab741) 74.34% compared to head (de45d5c) 84.59%.
Report is 142 commits behind head on master.

❗ Current head de45d5c differs from pull request most recent head 222cc4c. Consider uploading reports for the commit 222cc4c to get more accurate results

Files Patch % Lines
R/GitHost.R 79.14% 44 Missing ⚠️
R/EngineGraphQLGitLab.R 78.31% 36 Missing ⚠️
R/EngineRestGitHub.R 81.25% 33 Missing ⚠️
R/EngineGraphQLGitHub.R 79.23% 27 Missing ⚠️
R/EngineRestGitLab.R 75.75% 24 Missing ⚠️
R/GitStats.R 88.57% 16 Missing ⚠️
R/GQLQueryGitHub.R 70.83% 14 Missing ⚠️
R/GQLQueryGitLab.R 88.63% 10 Missing ⚠️
R/gitstats_functions.R 87.50% 5 Missing ⚠️

❗ 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.
📢 Have feedback on the report? Share it here.

@maciekbanas maciekbanas merged commit e42d640 into master Jan 8, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants