| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| ... | |
| |/
|
|
| |
It's mostly an internal thing anyway and can be removed in 3.0.0
|
| |\
| |
| | |
Improvements to HTTP requests
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Some arguments to `http_request` were being read
from kwargs, but kwargs is where this function
creates query parameters from, by default. In
the absence of a `query_parameters` param, the
function would construct URLs with query
parameters such as `retry_transient_errors=True`
despite those parameters having no meaning to
the API to which the request was sent.
This change names those arguments that are
specific to `http_request` so that they do not
end up as query parameters read from kwargs.
|
| | |
| |
| |
| | |
Fixes #1386
|
| |\ \
| | |
| | | |
fix: correct ProjectFile.decode() documentation
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
ProjectFile.decode() returns 'bytes' and not 'str'.
Update the method's doc-string and add a type-hint.
ProjectFile.decode() returns the result of a call to
base64.b64decode()
The docs for that function state it returns 'bytes':
https://docs.python.org/3/library/base64.html#base64.b64decode
Fixes: #1403
|
| |\ \ \
| | | |
| | | | |
Fix all issues reported by running: tox -e pep8 and enable pep8 as a linter check
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Local variable name is assigned to but never used
https://www.flake8rules.com/rules/F841.html
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
F401: Module imported but unused
https://www.flake8rules.com/rules/F401.html
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
E711: Comparison to none should be 'if cond is none:'
https://www.flake8rules.com/rules/E711.html
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
E712: Comparison to true should be 'if cond is true:' or 'if cond:'
https://www.flake8rules.com/rules/E712.html
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fixes to resolve errors for:
https://www.flake8rules.com/rules/E741.html
Do not use variables named 'I', 'O', or 'l' (E741)
https://www.flake8rules.com/rules/E742.html
Do not define classes named 'I', 'O', or 'l' (E742)
|
| | |/ /
|/| |
| | |
| | |
| | |
| | |
| | | |
If we want to update email, avatar and do not send email
confirmation change (`skip_reconfirmation` = True), `MultipartEncoder`
will try to encode everything except None and bytes. So it tries to encode bools.
Casting bool's values to their stringified int representation fix it.
|
| |\ \ \
| | | |
| | | | |
fix(types): prevent __dir__ in RestObject from producing duplicates
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | | |
chore: remove unused function sanitize_parameters()
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The function sanitize_parameters() was used when the v3 API was in
use. Since v3 API support has been removed there are no more users of
this function.
|
| |\ \ \ \ \
| | | | | |
| | | | | | |
fix: correct some type-hints in gitlab/mixins.py
|
| | |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Commit baea7215bbbe07c06b2ca0f97a1d3d482668d887 introduced type-hints
for gitlab/mixins.py.
After starting to add type-hints to gitlab/v4/objects/users.py
discovered a few errors.
Main error was using '=' instead of ':'. For example:
_parent = Optional[...] should be _parent: Optional[...]
Resolved those issues.
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
While adding type-hints mypy flagged this as an issue. The third
argument to register_custom_action is supposed to be a tuple. It was
being passed as a string rather than a tuple of strings.
|
| | | | | |
|
| | | | | |
|
| | |/ /
|/| | |
|
| |\ \ \
| | | |
| | | | |
feat(config): allow using a credential helper to lookup tokens
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | | |
chore: remove usage of getattr()
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Remove usage of getattr(self, "_update_uses_post", False)
Instead add it to class and set default value to False.
Add a tests that shows it is set to True for the
ProjectMergeRequestApprovalManager and ProjectApprovalManager classes.
|
| |\ \ \ \
| |_|_|/
|/| | | |
chore: have _create_attrs & _update_attrs be a namedtuple
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Convert _create_attrs and _update_attrs to use a NamedTuple
(RequiredOptional) to help with code readability. Update all code to
use the NamedTuple.
|
| |\ \ \ \
| |_|/ /
|/| | | |
Feat: cover all audit events
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| |_|/ /
|/| | | |
feat: add support for Project Package Files
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add ProjectPackageFile and the ability to list project package
package_files.
Fixes #1372
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | | |
Add the attributes: _create_attrs and _update_attrs to the RESTManager
class. This is so that we stop using getattr() if we don't need to.
This also helps with type-hints being available for these attributes.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Prior to commit 3727cbd21fc40b312573ca8da56e0f6cf9577d08
RESTManager._from_parent_attrs did not exist unless it was explicitly
set. But commit 3727cbd21fc40b312573ca8da56e0f6cf9577d08 set it to a
default value of {}.
So the checks using hasattr() were no longer valid.
Update the checks to check if RESTManager._from_parent_attrs has a
value.
|
