| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |/ |
|
| |\
| |
| | |
feat(api): return result from `SaveMixin.save()`
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Return the new object data when calling `SaveMixin.save()`.
Also remove check for `None` value when calling
`self.manager.update()` as that method only returns a dictionary.
Closes: #1081
|
| |\ \
| |/
|/| |
ci: don't fail CI if unable to upload the code coverage data
|
| |/
|
|
|
| |
If a CI job can't upload coverage results to codecov.com it causes the
CI to fail and code can't be merged.
|
| |\
| |
| | |
chore: ignore intermediate coverage artifacts
|
| |/ |
|
| |\
| |
| | |
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
|
| | |/
|/| |
|
| |\ \
| |/
|/| |
fix(api): services: add missing `lazy` parameter
|
| |/
|
|
|
|
|
|
| |
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
|
| |\
| |
| | |
fix: broken URL for FAQ about attribute-error-list
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
| |
The URL was missing a 'v' before the version number and thus the page
did not exist.
Previously the URL for python-gitlab 3.0.0 was:
https://python-gitlab.readthedocs.io/en/3.0.0/faq.html#attribute-error-list
Which does not exist.
Change it to:
https://python-gitlab.readthedocs.io/en/v3.0.0/faq.html#attribute-error-list
add the 'v' --------------------------^
|
| |
|
|
| |
See https://docs.gitlab.com/ee/api/group_access_tokens.html
|
| | |
|
| |\
| |
| | |
chore: add `pprint()` and `pformat()` methods to RESTObject
|
| | |
| |
| |
| |
| | |
This is useful in debugging and testing. As can easily print out the
values from an instance in a more human-readable form.
|
| |\ \
| | |
| | | |
test: add logging to `tests/functional/conftest.py`
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
I have found trying to debug issues in the functional tests can be
difficult. Especially when trying to figure out failures in the CI
running on Github.
Add logging to `tests/functional/conftest.py` to have a better
understanding of what is happening during a test run which is useful
when trying to troubleshoot issues in the CI.
|
| |\ \ \
| |/ /
|/| | |
fix: remove custom URL encoding
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |\ \
| | |
| | |
| | |
| | | |
python-gitlab/jlvillal/merge_request_merge_defaults
fix: remove default arguments for mergerequests.merge()
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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(cli): url-encode path components of the URL
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
In the CLI we need to make sure the components put into the path
portion of the URL are url-encoded. Otherwise they will be interpreted
as part of the path. For example can specify the project ID as a path,
but in the URL it must be url-encoded or it doesn't work.
Also stop adding the components of the path as query parameters in the
URL.
Closes: #783
Closes: #1498
|
| | | |
| | |
| | |
| | |
| | | |
Previously c8256a5933d745f70c7eea0a7d6230b51bac0fbc was done to fix
this but it missed two other failures.
|
| | | | |
|
| |/ / |
|
| |\ \
| | |
| | | |
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
|
| |\ \ \
| | | |
| | | | |
chore(dist): add docs *.md files to sdist
|
| | | |/
| |/|
| | |
| | | |
build_sphinx to fail due to setup.cfg warning-is-error
|
| |\ \ \
| |/ /
|/| | |
chore: fix missing comma
|
| |/ /
| |
| |
| |
| | |
There was a missing comma which meant the strings were concatenated
instead of being two separate strings.
|
| |\ \
| | |
| | | |
chore: add a stale workflow
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Use the stale action to close issues and pull-requests with no
activity.
Issues: It will mark them as stale after 60 days and then close
them once they have been stale for 15 days.
Pull-Requests: It will mark pull-requests as stale after 90 days and then close
them once they have been stale for 15 days.
https://github.com/actions/stale
Closes: #1649
|
| |\ \
| |/
|/| |
chore: add functional test of mergerequest.get()
|
| | |
| |
| |
| |
| |
| |
| | |
Add a functional test of test mergerequest.get() and
mergerequest.get(..., lazy=True)
Closes: #1425
|
| | | |
|
| | | |
|
| | | |
|
| |\ \
| |/
|/| |
chore: fix typo in MR documentation
|
| |/ |
|
| | |
|
| | |
|