summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge pull request #1319 from JohnVillalovos/jlvillal/import_starNejc Habjan2021-02-2557-114/+244
|\ \ | |/ |/| chore: remove usage of 'from ... import *'
| * chore: remove usage of 'from ... import *'John L. Villalovos2021-02-2357-114/+244
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In gitlab/v4/objects/*.py remove usage of: * from gitlab.base import * * from gitlab.mixins import * Change them to: * from gitlab.base import CLASS_NAME * from gitlab.mixins import CLASS_NAME Programmatically update code to explicitly import needed classes only. After the change the output of: $ flake8 gitlab/v4/objects/*py | grep 'REST\|Mixin' Is empty. Before many messages about unable to determine if it was a valid name.
* | Merge pull request #1327 from python-gitlab/feat/project-access-token-apiNejc Habjan2021-02-244-0/+167
|\ \ | | | | | | feat(projects): add project access token api
| * | test: don't add duplicate fixtureMax Wittig2021-02-241-26/+0
| | | | | | | | | Co-authored-by: Nejc Habjan <hab.nejc@gmail.com>
| * | feat(projects): add project access token apiMax Wittig2021-02-244-0/+193
| | |
* | | Merge pull request #1325 from JohnVillalovos/jlvillal/pep8Max Wittig2021-02-241-1/+5
|\ \ \ | |/ / |/| | fix: tox pep8 target, so that it can run
| * | fix: tox pep8 target, so that it can runJohn L. Villalovos2021-02-231-1/+5
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously running the pep8 target would fail as flake8 was not installed. Now install flake8 for the pep8 target. NOTE: Running the pep8 target fails as there are many warnings/errors. But it does allow us to run it and possibly work on reducing these warnings/errors in the future. In addition, add two checks to the ignore list as black takes care of formatting. The two checks added to the ignore list are: * E501: line too long * W503: line break before binary operator
* | chore(deps): update gitlab/gitlab-ce docker tag to v13.9.1-ce.0Renovate Bot2021-02-241-1/+1
|/
* Merge pull request #1322 from JohnVillalovos/jlvillal/missing_varsNejc Habjan2021-02-242-4/+8
|\ | | | | fix: undefined name errors in v4 objects
| * fix: undefined name errorsJohn L. Villalovos2021-02-232-4/+8
|/ | | | Discovered that there were some undefined names.
* Merge pull request #1321 from JohnVillalovos/jlvillal/remove_cruftNejc Habjan2021-02-231-18/+0
|\ | | | | chore: remove unused function _construct_url()
| * chore: remove unused function _construct_url()John L. Villalovos2021-02-231-18/+0
|/ | | | | | The function _construct_url() was used by the v3 API. All usage of the function was removed in commit fe89b949922c028830dd49095432ba627d330186
* Merge pull request #1299 from JohnVillalovos/mypyNejc Habjan2021-02-237-32/+55
|\ | | | | Enable mypy type checking and add type hints to gitlab/base.py
| * chore: add type hints to gitlab/base.pyJohn L. Villalovos2021-02-221-29/+34
| |
| * feat: add an initial mypy test to tox.iniJohn L. Villalovos2021-02-226-3/+21
|/ | | | Add an initial mypy test to test gitlab/base.py and gitlab/__init__.py
* Merge pull request #1318 from JohnVillalovos/jlvillal/testingNejc Habjan2021-02-221-16/+16
|\ | | | | chore: remove usage of 'from ... import *' in client.py
| * chore: remove usage of 'from ... import *' in client.pyJohn L. Villalovos2021-02-221-16/+16
|/ | | | | | | | | | | | | | | | In gitlab/client.py remove usage of: * from gitlab.const import * * from gitlab.exceptions import * Change them to: * import gitlab.const * import gitlab.exceptions Update code to explicitly reference things in gitlab.const and gitlab.exceptions A flake8 run no longer lists any undefined variables. Before it listed possible undefined variables.
* Merge pull request #1310 from JohnVillalovos/jlvillal/v4_onlyNejc Habjan2021-02-222-10/+29
|\ | | | | chore: explicitly import gitlab.v4.objects/cli
| * chore: explicitly import gitlab.v4.objects/cliJohn L. Villalovos2021-02-212-10/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As we only support the v4 Gitlab API, explicitly import gitlab.v4.objects and gitlab.v4.clie instead of dynamically importing it depending on the API version. This has the added benefit of mypy being able to type check the Gitlab __init__() function as currently it will fail if we enable type checking of __init__() it will fail. Also, this also helps by not confusing tools like pyinstaller/cx_freeze with dynamic imports so you don't need hooks for standalone executables. And according to https://docs.gitlab.com/ee/api/, "GraphQL co-exists with the current v4 REST API. If we have a v5 API, this should be a compatibility layer on top of GraphQL."
* | chore(deps): update gitlab/gitlab-ce docker tag to v13.9.0-ce.0Renovate Bot2021-02-221-1/+1
|/
* Merge pull request #1316 from JohnVillalovos/jlvillal/test_waitNejc Habjan2021-02-212-3/+10
|\ | | | | test: extend wait timeout for test_delete_user()
| * fix: extend wait timeout for test_delete_user()John L. Villalovos2021-02-212-3/+10
|/ | | | | | | | | | | | | | | | | | | | | | | Have been seeing intermittent failures of the test_delete_user() functional test. Have made the following changes to hopefully resolve the issue and if it still fails to know better why the failure occurred. * Extend the wait timeout for test_delete_user() from 30 to 60 tries of 0.5 seconds each. * Modify wait_for_sidekiq() to return True if sidekiq process terminated. Return False if the timeout expired. * Modify wait_for_sidekiq() to loop through all processes instead of assuming there is only one process. If all processes are not busy then return. * Modify wait_for_sidekiq() to sleep at least once before checking for processes being busy. * Check for True being returned in test_delete_user() call to wait_for_sidekiq()
* Merge pull request #1314 from python-gitlab/feat/release-linksMax Wittig2021-02-2112-83/+329
|\ | | | | feat: add release links API support
| * docs(api): add release links API docsNejc Habjan2021-02-213-33/+78
| |
| * test(api): add functional test for release links APINejc Habjan2021-02-213-35/+70
| |
| * chore(objects): make Project refreshableNejc Habjan2021-02-211-1/+1
| | | | | | | | Helps getting the real state of the project from the server.
| * feat(objects): add Release Links API supportNejc Habjan2021-02-216-14/+180
| |
* | Merge pull request #1311 from JohnVillalovos/jlvillal/fix_functionalNejc Habjan2021-02-211-1/+1
|\ \ | |/ |/| fix: test_update_group() dependency on ordering
| * fix: test_update_group() dependency on orderingJohn L. Villalovos2021-02-201-1/+1
|/ | | | | | | | Since there are two groups we can't depend on the one we changed to always be the first one returned. Instead fetch the group we want and then test our assertion against that group.
* Merge pull request #1307 from python-gitlab/renovate/docker-compose-1.xNejc Habjan2021-02-201-1/+1
|\ | | | | chore(deps): update dependency docker-compose to v1.28.4
| * chore(deps): update dependency docker-compose to v1.28.4Renovate Bot2021-02-181-1/+1
| |
* | Merge pull request #1308 from Sineaggi/add-project-audit-endpointNejc Habjan2021-02-203-0/+104
|\ \ | | | | | | feat: add project audit endpoint
| * | chore(tests): remove unused URL segmentNejc Habjan2021-02-201-2/+2
| | |
| * | feat: add project audit endpointClayton Walker2021-02-193-0/+104
| |/
* | Merge pull request #1301 from JohnVillalovos/refactor_jlvillalNejc Habjan2021-02-203-834/+860
|\ \ | |/ |/| refactor: move Gitlab and GitlabList to gitlab/client.py
| * refactor: move Gitlab and GitlabList to gitlab/client.pyJohn L. Villalovos2021-02-183-834/+860
|/ | | | | | | | Move the classes Gitlab and GitlabList from gitlab/__init__.py to the newly created gitlab/client.py file. Update one test case that was depending on requests being defined in gitlab/__init__.py
* Merge pull request #1305 from python-gitlab/renovate/docker-compose-1.xNejc Habjan2021-02-181-1/+1
|\ | | | | chore(deps): update dependency docker-compose to v1.28.3
| * chore(deps): update dependency docker-compose to v1.28.3Renovate Bot2021-02-171-1/+1
| |
* | Merge pull request #1304 from python-gitlab/feat/personal-access-token-apiNejc Habjan2021-02-185-0/+94
|\ \ | |/ |/| feat: add personal access token API
| * feat: add personal access token APIfeat/personal-access-token-apiMax Wittig2021-02-175-0/+94
|/ | | | See: https://docs.gitlab.com/ee/api/personal_access_tokens.html
* Merge pull request #1300 from JohnVillalovos/remove_cruftNejc Habjan2021-02-161-2/+0
|\ | | | | chore: remove unused ALLOWED_KEYSET_ENDPOINTS variable
| * chore: remove unused ALLOWED_KEYSET_ENDPOINTS variableJohn L. Villalovos2021-02-151-2/+0
| | | | | | | | | | | | | | | | | | The variable ALLOWED_KEYSET_ENDPOINTS was added in commit f86ef3bbdb5bffa1348a802e62b281d3f31d33ad. Then most of that commit was removed in commit e71fe16b47835aa4db2834e98c7ffc6bdec36723, but ALLOWED_KEYSET_ENDPOINTS was missed.
* | Merge pull request #1303 from python-gitlab/renovate/sphinx-3.xNejc Habjan2021-02-161-1/+1
|\ \ | |/ |/| chore(deps): update dependency sphinx to v3.5.1
| * chore(deps): update dependency sphinx to v3.5.1Renovate Bot2021-02-161-1/+1
|/
* Merge pull request #1271 from ↵Nejc Habjan2021-02-151-2/+2
|\ | | | | | | | | allcloud-jonathan/feature/honor-bool-for-delete-source fix: honor parameter value passed
| * fix: honor parameter value passedJonathan Vogt2021-02-151-2/+2
| | | | | | | | | | | | Gitlab allows setting the defaults for MR to delete the source. Also the inline help of the CLI suggest that a boolean is expected, but no matter what value you set, it will always delete.
* | Merge pull request #1298 from JohnVillalovos/masterMax Wittig2021-02-151-4/+1
|\ \ | |/ |/| Remove Python 2 code
| * chore: remove Python 2 codeJohn L. Villalovos2021-02-151-4/+1
|/ | | | | | | httplib is a Python 2 library. It was renamed to http.client in Python 3. https://docs.python.org/2.7/library/httplib.html
* Merge pull request #1288 from python-gitlab/refactor/split-objectsMax Wittig2021-02-1557-5850/+6715
|\ | | | | refactor(v4): split objects and managers per API resource
| * refactor(api): explicitly export classes for star importsrefactor/split-objectsNejc Habjan2021-02-0756-9/+557
| |