Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore: create new ArrayAttribute class#1866

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
nejch merged 1 commit intomainfromjlvillal/arrays_2
Jan 31, 2022
Merged

Conversation

@JohnVillalovos
Copy link
Member

Create a new ArrayAttribute class. This is to indicate types which are
sent to the GitLab server as arrays
https://docs.gitlab.com/ee/api/#array

At this stage it is identical to the CommaSeparatedListAttribute class
but will be used later to support the array types sent to GitLab.

This is the second step in a series of steps of our goal to add full
support for the GitLab API data types[1]:

  • array
  • hash
  • array of hashes

Step one was: commit5127b15

[1]https://docs.gitlab.com/ee/api/#encoding-api-parameters-of-array-and-hash-types

Related:#1698

Create a new ArrayAttribute class. This is to indicate types which aresent to the GitLab server as arrayshttps://docs.gitlab.com/ee/api/#arrayAt this stage it is identical to the CommaSeparatedListAttribute classbut will be used later to support the array types sent to GitLab.This is the second step in a series of steps of our goal to add fullsupport for the GitLab API data types[1]:  * array  * hash  * array of hashesStep one was: commit5127b15[1]https://docs.gitlab.com/ee/api/#encoding-api-parameters-of-array-and-hash-typesRelated:#1698
@codecov-commenter
Copy link

Codecov Report

Merging#1866 (a57334f) intomain (7a13b9b) willincrease coverage by0.00%.
The diff coverage is100.00%.

@@           Coverage Diff           @@##             main    #1866   +/-   ##=======================================  Coverage   92.35%   92.35%           =======================================  Files          77       77             Lines        4854     4856    +2     =======================================+ Hits         4483     4485    +2  Misses        371      371
FlagCoverage Δ
cli_func_v481.40% <100.00%> (+<0.01%)⬆️
py_func_v480.29% <100.00%> (+<0.01%)⬆️
unit83.31% <100.00%> (+<0.01%)⬆️

Flags with carried forward coverage won't be shown.Click here to find out more.

Impacted FilesCoverage Δ
gitlab/v4/objects/merge_requests.py84.78% <ø> (ø)
gitlab/v4/objects/settings.py95.45% <ø> (ø)
gitlab/types.py96.66% <100.00%> (+0.23%)⬆️
gitlab/v4/objects/groups.py88.65% <100.00%> (ø)
gitlab/v4/objects/issues.py89.02% <100.00%> (ø)
gitlab/v4/objects/members.py94.73% <100.00%> (ø)
gitlab/v4/objects/milestones.py100.00% <100.00%> (ø)
gitlab/v4/objects/projects.py88.09% <100.00%> (ø)
gitlab/v4/objects/users.py95.90% <100.00%> (ø)

"iids":types.CommaSeparatedListAttribute,
"labels":types.CommaSeparatedListAttribute,
}
_types= {"iids":types.ArrayAttribute,"labels":types.CommaSeparatedListAttribute}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

lol, this is infuriating:
https://docs.gitlab.com/ee/api/issues.html#list-issues

GET /issues?iids[]=42&iids[]=43GET /issues?labels=foo,bar

😁 thanks for going through all the endpoints. I hope the API docs are accurate.

@nejchnejch merged commit7646360 intomainJan 31, 2022
@nejchnejch deleted the jlvillal/arrays_2 branchJanuary 31, 2022 23:04
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nejchnejchnejch left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@JohnVillalovos@codecov-commenter@nejch

[8]ページ先頭

©2009-2025 Movatter.jp