summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | fix: remove custom URL encodingJohn L. Villalovos2022-01-088-22/+50
|/ / | | | | | | | | | | | | | | | | | | 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
* | Merge pull request #1818 from ↵Nejc Habjan2022-01-092-5/+7
|\ \ | | | | | | | | | | | | python-gitlab/jlvillal/merge_request_merge_defaults fix: remove default arguments for mergerequests.merge()
| * | fix: remove default arguments for mergerequests.merge()John L. Villalovos2022-01-082-5/+7
|/ / | | | | | | | | | | | | | | | | | | 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
* | Merge pull request #1790 from python-gitlab/jlvillal/parent_attrsNejc Habjan2022-01-084-18/+71
|\ \ | | | | | | fix(cli): url-encode path components of the URL
| * | fix(cli): url-encode path components of the URLJohn L. Villalovos2022-01-084-13/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | chore: fix functional test failure if config presentJohn L. Villalovos2022-01-081-5/+4
| | | | | | | | | | | | | | | Previously c8256a5933d745f70c7eea0a7d6230b51bac0fbc was done to fix this but it missed two other failures.
* | | docs(cli): make examples more easily navigable by generating TOCNejc Habjan2022-01-081-0/+27
| | |
* | | chore(docs): use admonitions consistentlyNejc Habjan2022-01-082-3/+3
|/ /
* | Merge pull request #1809 from python-gitlab/jlvillal/list_apiNejc Habjan2022-01-081-8/+11
|\ \ | | | | | | fix: change to `http_list` for some ProjectCommit methods
| * | fix: change to `http_list` for some ProjectCommit methodsJohn L. Villalovos2022-01-081-8/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge pull request #1813 from derekschrock/missing-distNejc Habjan2022-01-081-1/+1
|\ \ \ | | | | | | | | chore(dist): add docs *.md files to sdist
| * | | chore(dist): add docs *.md files to sdistDerek Schrock2022-01-071-1/+1
| | |/ | |/| | | | | | | build_sphinx to fail due to setup.cfg warning-is-error
* | | Merge pull request #1814 from python-gitlab/jlvillal/missing_commaNejc Habjan2022-01-081-1/+2
|\ \ \ | |/ / |/| | chore: fix missing comma
| * | chore: fix missing commajlvillal/missing_commaJohn L. Villalovos2022-01-071-1/+2
|/ / | | | | | | | | There was a missing comma which meant the strings were concatenated instead of being two separate strings.
* | Merge pull request #1789 from python-gitlab/jlvillal/staleNejc Habjan2022-01-071-0/+40
|\ \ | | | | | | chore: add a stale workflow
| * | chore: add a stale workflowjlvillal/staleJohn L. Villalovos2022-01-061-0/+40
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Merge pull request #1803 from python-gitlab/jlvillal/test_1425Nejc Habjan2022-01-061-0/+16
|\ \ | |/ |/| chore: add functional test of mergerequest.get()
| * chore: add functional test of mergerequest.get()jlvillal/test_1425John L. Villalovos2022-01-031-0/+16
| | | | | | | | | | | | | | Add a functional test of test mergerequest.get() and mergerequest.get(..., lazy=True) Closes: #1425
* | chore: add temporary banner for v3Nejc Habjan2022-01-051-1/+4
| |
* | chore: release v3.0.0v3.0.0github-actions2022-01-052-1/+68
| |
* | docs: switch to Furo and refresh introduction pagesNejc Habjan2022-01-048-269/+216
| |
* | Merge pull request #1804 from mlegner/patch-1Nejc Habjan2022-01-041-1/+1
|\ \ | |/ |/| chore: fix typo in MR documentation
| * chore: fix typo in MR documentationMarkus Legner2022-01-041-1/+1
|/
* chore(deps): update dependency argcomplete to v2Renovate Bot2022-01-033-3/+3
|
* chore(deps): update dependency requests to v2.27.0Renovate Bot2022-01-032-2/+2
|
* Merge pull request #1800 from python-gitlab/jlvillal/dot_branchNejc Habjan2022-01-041-0/+17
|\ | | | | chore: add test case to show branch name with period works
| * chore: add test case to show branch name with period worksjlvillal/dot_branchJohn L. Villalovos2022-01-031-0/+17
| | | | | | | | | | | | | | Add a test case to show that a branch name with a period can be fetched with a `get()` Closes: #1715
* | chore(deps): update typing dependenciesRenovate Bot2022-01-032-4/+4
| |
* | Merge pull request #1799 from python-gitlab/renovate/mypy-0.xNejc Habjan2022-01-041-1/+1
|\ \ | |/ |/| chore(deps): update dependency mypy to v0.930
| * chore(deps): update dependency mypy to v0.930Renovate Bot2022-01-041-1/+1
|/
* feat(docker): remove custom entrypoint from imageNejc Habjan2022-01-033-37/+26
| | | | | This is no longer needed as all of the configuration is handled by the CLI and can be passed as arguments.
* feat(cli): allow options from args and environment variablesNejc Habjan2022-01-027-25/+421
| | | | | | | | BREAKING-CHANGE: The gitlab CLI will now accept CLI arguments and environment variables for its global options in addition to configuration file options. This may change behavior for some workflows such as running inside GitLab CI and with certain environment variables configured.
* chore(deps): upgrade mypy pre-commit hookNejc Habjan2022-01-021-1/+1
|
* Merge pull request #1792 from python-gitlab/jlvillal/cli_testNejc Habjan2022-01-021-4/+8
|\ | | | | chore: fix functional test failure if config present
| * chore: fix functional test failure if config presentjlvillal/cli_testJohn L. Villalovos2022-01-011-4/+8
|/ | | | | | | Fix functional test failure if config present and configured with token. Closes: #1791
* Merge pull request #1773 from python-gitlab/jlvillal/paginationNejc Habjan2021-12-315-23/+35
|\ | | | | fix: handle situation where gitlab.com does not return values
| * fix: handle situation where GitLab does not return valuesjlvillal/paginationJohn L. Villalovos2021-12-305-23/+35
|/ | | | | | | | | | | | | | | | If a query returns more than 10,000 records than the following values are NOT returned: x.total_pages x.total Modify the code to allow no value to be set for these values. If there is not a value returned the functions will now return None. Update unit test so no longer `xfail` https://docs.gitlab.com/ee/user/gitlab_com/index.html#pagination-response-headers Closes #1686
* test: reproduce missing pagination headers in testsNejc Habjan2021-12-301-0/+30
|
* test: drop httmock dependency in test_gitlab.pyNejc Habjan2021-12-301-45/+61
|
* Merge pull request #1783 from python-gitlab/jlvillal/sidekiqNejc Habjan2021-12-291-0/+30
|\ | | | | chore: ensure reset_gitlab() succeeds
| * chore: ensure reset_gitlab() succeedsJohn L. Villalovos2021-12-281-0/+30
|/ | | | | | | | | Ensure reset_gitlab() succeeds by waiting to make sure everything has been deleted as expected. If the timeout is exceeded fail the test. Not using `wait_for_sidekiq` as it didn't work. During testing I didn't see any sidekiq processes as being busy even though not everything was deleted.
* Merge pull request #1782 from python-gitlab/jlvillal/repository_func_testsNejc Habjan2021-12-281-0/+4
|\ | | | | chore: skip a functional test if not using >= py3.9
| * chore: skip a functional test if not using >= py3.9jlvillal/repository_func_testsJohn L. Villalovos2021-12-261-0/+4
| | | | | | | | | | One of the tests requires Python 3.9 or higher to run. Mark the test to be skipped if running Python less than 3.9.
* | Merge pull request #1781 from python-gitlab/jlvillal/docker_composeNejc Habjan2021-12-281-1/+1
|\ \ | |/ |/| chore: update version in docker-compose.yml
| * chore: update version in docker-compose.ymljlvillal/docker_composeJohn L. Villalovos2021-12-261-1/+1
|/ | | | | | | | | | | When running with docker-compose on Ubuntu 20.04 I got the error: $ docker-compose up ERROR: The Compose file './docker-compose.yml' is invalid because: networks.gitlab-network value Additional properties are not allowed ('name' was unexpected) Changing the version in the docker-compose.yml file fro '3' to '3.5' resolved the issue.
* docs: correct documentation for updating discussion noteHailiang2021-12-251-1/+1
| | | | Closes #1777
* Merge pull request #1774 from python-gitlab/jlvillal/doc_artifactsNejc Habjan2021-12-251-0/+5
|\ | | | | chore: generate artifacts for the docs build in the CI
| * chore: generate artifacts for the docs build in the CIjlvillal/doc_artifactsJohn L. Villalovos2021-12-221-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | When building the docs store the created documentation as an artifact so that it can be viewed. This will create a html-docs.zip file which can be downloaded containing the contents of the `build/sphinx/html/` directory. It can be downloaded, extracted, and then viewed. This can be useful in reviewing changes to the documentation. See https://github.com/actions/upload-artifact for more information on how this works.
* | Merge pull request #1776 from python-gitlab/jlvillal/rebase_in_progressNejc Habjan2021-12-254-2/+15
|\ \ | |/ |/| Add some docs for getting the status of a merge_request rebase
| * chore: add and document optional parameters for get MRjlvillal/rebase_in_progressJohn L. Villalovos2021-12-232-0/+13
| | | | | | | | | | | | | | Add and document (some of the) optional parameters that can be done for a `project.merge_requests.get()` Closes #1775