| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Create return type-hints for `RESTObject.get_id()` and
`RESTObject.encoded_id`. Previously was saying they return Any. Be
more precise in saying they can return either: None, str, or int.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
| |
Updated a few remaining usages of `self.id` to use `self.encoded_id`
as for the most part we shouldn't be using `self.id`
There are now only a few (4 lines of code) remaining uses of
`self.id`, most of which seem that they should stay that way.
|
| |\
| |
| | |
chore(objects): use `self.encoded_id` where could be a string
|
| | |
| |
| |
| |
| | |
Updated a few remaining usages of `self.id` to use `self.encoded_id`
where it could be a string value.
|
| | |
| |
| | |
Co-authored-by: John Villalovos <john@sodarock.com>
|
| |/ |
|
| |\
| |
| | |
fix: use url-encoded ID in all paths
|
| | |
| |
| |
| |
| |
| | |
utils.EncodedId() has basically the same functionalityy of using
utils._url_encode(). So remove utils._url_encode() as we don't need
it.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Add EncodedId string class. This class returns a URL-encoded string
but ensures it will only URL-encode it once even if recursively
called.
Also added some functional tests of 'lazy' objects to make sure they
work.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Make sure all usage of the ID in the URL path is encoded. Normally it
isn't an issue as most IDs are integers or strings which don't contain
a slash ('/'). But when the ID is a string with a slash character it
will break things.
Add a test case that shows this fixes wikis issue with subpages which
use the slash character.
Closes: #1079
|
| |\ \
| | |
| | | |
fix: members: use new *All objects for *AllManager managers
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Change it so that:
GroupMemberAllManager uses GroupMemberAll object
ProjectMemberAllManager uses ProjectMemberAll object
Create GroupMemberAll and ProjectMemberAll objects that do not support
any Mixin type methods. Previously we were using GroupMember and
ProjectMember which support the `save()` and `delete()` methods but
those methods will not work with objects retrieved using the
`/members/all/` API calls.
`list()` API calls: [1]
GET /groups/:id/members/all
GET /projects/:id/members/all
`get()` API calls: [2]
GET /groups/:id/members/all/:user_id
GET /projects/:id/members/all/:user_id
Closes: #1825
Closes: #848
[1] https://docs.gitlab.com/ee/api/members.html#list-all-members-of-a-group-or-project-including-inherited-and-invited-members
[2] https://docs.gitlab.com/ee/api/members.html#get-a-member-of-a-group-or-project-including-inherited-and-invited-members
|
| | |/
|/| |
|
| |/
|
|
|
|
|
|
| |
Commit 8da0b758c589f608a6ae4eeb74b3f306609ba36d added the `lazy`
parameter to the services `get()` method but missed then using the
`lazy` parameter when it called `super(...).get(...)`
Closes: #1828
|
| |
|
|
| |
See https://docs.gitlab.com/ee/api/group_access_tokens.html
|
| |
|
|
|
|
|
|
|
|
| |
We were using `str.replace()` calls to take care of URL encoding
issues.
Switch them to use our `utils._url_encode()` function which itself uses
`urllib.parse.quote()`
Closes: #1356
|
| |
|
|
|
|
|
|
|
|
| |
The arguments `should_remove_source_branch` and
`merge_when_pipeline_succeeds` are optional arguments. We should not
be setting any default value for them.
https://docs.gitlab.com/ee/api/merge_requests.html#accept-mr
Closes: #1750
|
| |\
| |
| | |
fix: change to `http_list` for some ProjectCommit methods
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Fix the type-hints and use `http_list()` for the ProjectCommits methods:
- diff()
- merge_requests()
- refs()
This will enable using the pagination support we have for lists.
Closes: #1805
Closes: #1231
|
| |/
|
|
|
| |
There was a missing comma which meant the strings were concatenated
instead of being two separate strings.
|
| |
|
|
|
|
|
| |
Add and document (some of the) optional parameters that can be done
for a `project.merge_requests.get()`
Closes #1775
|
| | |
|
| |
|
|
|
|
|
| |
There is an optional `squash_option` parameter which can be used when
creating Projects and UserProjects.
Closes #1744
|
| |
|
|
| |
It is not yet available upstream.
|
| | |
|
| |\
| |
| | |
chore: add get() methods for GetWithoutIdMixin based classes
|
| | |
| |
| |
| |
| |
| |
| | |
Add the get() methods for the GetWithoutIdMixin based classes.
Update the tests/meta/test_ensure_type_hints.py tests to check to
ensure that the get methods are defined with the correct return type.
|
| |/ |
|
| |
|
|
|
|
| |
Initial pylint check is added. A LONG list of disabled checks is also
added. In the future we should work through the list and resolve the
errors or disable them on a more granular level.
|
| |
|
|
| |
Enable 'warn_redundant_casts'for mypy and resolve one issue.
|
| | |
|
| | |
|
| |\
| |
| | |
feat(api): add support for epic notes
|
| | |
| |
| |
| |
| |
| | |
Added support for notes on group epics
Signed-off-by: Raimund Hook <raimund.hook@exfo.com>
|
| |/
|
|
|
|
| |
Add support for `projects.groups.list()` endpoint.
Closes #1717
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
| |
Fix classes which don't have correct 'get()' methods for classes
derived from GetMixin.
Add a unit test which verifies that classes have the correct return
type in their 'get()' method.
|