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.

Sign up for GitHub

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

Jump to bottom

2.1.1 patch release #518

Merged
merged 110 commits into from
Oct 29, 2024
Merged

2.1.1 patch release #518

merged 110 commits into from
Oct 29, 2024

Conversation

maciekbanas
Copy link
Member

No description provided.

maciekbanas and others added 30 commits October 1, 2024 08:14
@maciekbanas
Mock commit responses.
bbe8743
@maciekbanas
Mock tests for GitHub releases.
9d22d48
@maciekbanas
Mock tests for GitLab releases.
8ad995c
@maciekbanas
Mock repos responses for tests.
5588b8b
@maciekbanas
Mock repos urls, users and GitLab commits method for geting authors o…
0a89aba 
…f commits.
@maciekbanas
Mocking API responses in tests, improve test objects, make more use o…
af20413 
…f them, small improvements to code.
@maciekbanas
Fix function.
2730946
@maciekbanas
Mock files structure tests for GitLab.
c9fd7a5
@maciekbanas
Mock some responses.
057c64b
@maciekbanas
Merge pull request #490 from r-world-devs/481-mock-api-responses-2
c6fd4a7 
Mock further API responses
@maciekbanas
Fix error by getting back from |> to %>% operator and refactor fo…
0822050 
…r loops to purrr iterations. The `|>` does not support `.` argument in pipelines.
@maciekbanas
Add test with large search response.
bd6149f
@maciekbanas
Fix when r_package_usage output is empty.
b3e8be2
@maciekbanas
Limit expectation to one object.
30b7997
@maciekbanas
Add test when empty response.
6123610
@maciekbanas
New snap.
ef76025
@maciekbanas
Bump version, add NEWS.
ce37af8
@maciekbanas
Merge pull request #495 from r-world-devs/491-fix-search_response
8b751aa 
491 fix search response
@maciekbanas
Add getting GitHub repositories for individual user. Together with ne…
d8cc7f7 
…w feature a new method was introduced to check if owner is a user or an organization.
@maciekbanas
Bump version.
eb965dd
@maciekbanas
Prepare test for helper setting type attributes.
6477c7e
@maciekbanas
Add getting GitLab repos for user.
c39069e
@maciekbanas
Update NEWS.
a06f371
@maciekbanas
Fix code and tests.
325c05e
@maciekbanas
Update docs.
6186fe5
@maciekbanas
Add tests to cover new feature.
4fd12cc
@maciekbanas
Adjust getting files to new feature.
e1cbbcc
@maciekbanas
Add publishing of code coverage.
b6b214e
@maciekbanas
Try fix workflow.
7059be5
@maciekbanas
Merge pull request #500 from r-world-devs/496-add-code-coverage-job
3b3e29e 
496 add code coverage job
maciekbanas and others added 16 commits October 18, 2024 13:40
@maciekbanas
Merge pull request #514 from r-world-devs/512-bump-version-to-211
901b854 
Bump version, update NEWS.
@maciekbanas
Adjust package to submit to CRAN.
f3ee13f
@maciekbanas
Remove snapshots so tests can pass on CRAN. Tests with snapshots are …
01a4842 
…skipped on CRAN which results in case of GitStats for some tests not passing outputs to test_mocker and in result failutre of following up tests.
@maciekbanas
Follow up and try to fix coverage patch job.
ae3f066
@maciekbanas
Fix tests workflow for CRAN.
a02e1b5
@maciekbanas
Add test to pass coverage package job.
8064c53
@maciekbanas
Add test to meet coverage job.
7afaf92
@maciekbanas
Fix note from CRAN submission.
ac7cbd4
@maciekbanas
Add test to meet coverage job.
19f905e
@maciekbanas
Fix note from CRAN submission.
deea28d
@maciekbanas
Fix version bump check job. Should not run only when DESCRIPTION is m…
00451d3 
…odified.
@maciekbanas
Bump version.
7ae461e
@maciekbanas
Add test to meet coverage job.
68c6629
@maciekbanas
Merge pull request #515 from r-world-devs/319-tweak-gitstats-for-cran…
d7847cc 
…-release

319 tweak gitstats for cran release
@maciekbanas
Bump version.
c31ea7b
@maciekbanas
Merge pull request #517 from r-world-devs/516-bump-version-211
1cc49e4 
Bump version.
@maciekbanas maciekbanas self-assigned this Oct 21, 2024
@codecov Codecov
Copy link

codecov bot commented Oct 21, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 83.98577% with 135 lines in your changes missing coverage. Please review.

Project coverage is 75.01%. Comparing base (f0ab741) to head (a12bf97).
Report is 498 commits behind head on master.

Files with missing lines Patch % Lines
R/EngineRestGitLab.R 60.37% 42 Missing ⚠️
R/GitHost.R 76.81% 32 Missing ⚠️
R/EngineGraphQLGitLab.R 86.12% 24 Missing ⚠️
R/GitStats.R 86.17% 13 Missing ⚠️
R/EngineGraphQLGitHub.R 93.28% 10 Missing ⚠️
R/EngineRestGitHub.R 90.32% 6 Missing ⚠️
R/GQLQueryGitLab.R 55.55% 4 Missing ⚠️
R/gitstats_functions.R 50.00% 2 Missing ⚠️
R/EngineRest.R 95.65% 1 Missing ⚠️
R/GitHostGitLab.R 96.77% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #518      +/-   ##
==========================================
+ Coverage   74.34%   75.01%   +0.66%     
==========================================
  Files          11       17       +6     
  Lines         998     3146    +2148     
==========================================
+ Hits          742     2360    +1618     
- Misses        256      786     +530

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

maciekbanas and others added 4 commits October 21, 2024 08:32
@maciekbanas
Set proper token for pkgdown. After fixing checks for token scopes it…
37349ce 
… occured that default GITHUB_TOKEN is not sufficient. Therefore a new TEST_GITHUB_PAT was set in secret variables for the jobs.
@maciekbanas
Spaces.
0505284
@maciekbanas
Bump version.
2748201
@maciekbanas
Merge pull request #520 from r-world-devs/519-fix-github_pat-secret-f…
4df8cd6 
…or-pkgdown-job

519 fix GitHub pat secret for pkgdown job
@maciekbanas maciekbanas marked this pull request as ready for review October 23, 2024 08:04
@maciekbanas maciekbanas merged commit 45a26d6 into master Oct 29, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcinkowskak marcinkowskak marcinkowskak approved these changes

Assignees

@maciekbanas maciekbanas

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maciekbanas @marcinkowskak