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

refactor: migrate project:archive feature to use python-gitlab library #622

Merged
merged 3 commits into from
Oct 27, 2023

Conversation

amimas
Copy link
Collaborator

@amimas amimas commented Oct 26, 2023

This PR builds on top of #582 that provided the setup to use python-gitlab library. Refactored the project:archive feature's implementation to use the python-gitlab library instead of internal methods that called GitLab's API endpoint.

closes #621

Moved the Gitlab object created from GitlabWrapper to the AbstractProcessor
class so that the object is available to all other processors. Then updated
project processor code to use the Gitlab object, which comes from the
python-gitlab library, for performing project archive and unarchive.

Removed the internal API/method that was previously used for calling the
GitLab's API for project archive and unarchive purpose.
@amimas amimas requested a review from gdubicki October 26, 2023 02:17
@amimas amimas had a problem deploying to Integrate Pull Request October 26, 2023 02:17 — with GitHub Actions Failure
@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #622 (62adabe) into main (2ea3dcd) will increase coverage by 0.13%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #622      +/-   ##
==========================================
+ Coverage   83.50%   83.63%   +0.13%     
==========================================
  Files          70       70              
  Lines        2746     2744       -2     
==========================================
+ Hits         2293     2295       +2     
+ Misses        453      449       -4     
Files Coverage Δ
gitlabform/gitlab/projects.py 53.76% <ø> (+0.15%) ⬆️
gitlabform/processors/abstract_processor.py 86.17% <100.00%> (+0.45%) ⬆️
gitlabform/processors/project/project_processor.py 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

@amimas amimas temporarily deployed to Integrate Pull Request October 26, 2023 08:25 — with GitHub Actions Inactive
@amimas amimas merged commit 7dbd5c0 into gitlabform:main Oct 27, 2023
23 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.

Migrate project:archive feature to use python-gitlab library
2 participants