Skip to content

Conversation

@konfetka1989
Copy link

No description provided.

Dmitry Klochkov added 2 commits September 21, 2021 20:31
1) At the beginning, get a list of only top level groups. This lets us
   get rid of checking of the parent_id field and initialize the
   progress bar with a correct total value.

2) Invoke gitlab.groups.get() with the lazy=True parameter. This lets us
   not make an additional API call to get information which we already
   have.
If a gitlab user has access to a large amount of groups, it may take
much time to load an entire tree.  The option --root-group lets the user
specify an exact group and just use it as the root instead of waiting
for the entire tree to be loaded and filtered.
@ezbz ezbz closed this Jun 30, 2024
ezbz pushed a commit that referenced this pull request Jun 30, 2024
…ral approved PRs that broke them

- remove incompatible root group PR: #89
- integrate useful conflicting PR by @gsobczyk for excluding shared projects
- added support for incorrect PR (#119) by adding the get_all parameter to projects (the PR has it on groups and subgroups but it doesn't work since this parameter doesn't exist on these methods https://github.com/ezbz/gitlabber/pull/119/files) so the original PR cannot be merged without breaking the project
@ezbz ezbz mentioned this pull request Jun 30, 2024
ezbz added a commit that referenced this pull request Jun 30, 2024
* update library dependencies

* - recover passing state for unit / integration / e2e tests after several approved PRs that broke them
- remove incompatible root group PR: #89
- integrate useful conflicting PR by @gsobczyk for excluding shared projects
- added support for incorrect PR (#119) by adding the get_all parameter to projects (the PR has it on groups and subgroups but it doesn't work since this parameter doesn't exist on these methods https://github.com/ezbz/gitlabber/pull/119/files) so the original PR cannot be merged without breaking the project

* add actually working support for attempted PR: #119
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.

2 participants