-
Notifications
You must be signed in to change notification settings - Fork 49
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Integration][GitLab] Fix few bugs #223
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌊 left few comments
Project, | ||
] | ||
], | ||
[Any], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you open a task on me to fix this typing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure this is the correct typing though.
The paging shouldn't care what the validation function it gets; Each validation function should define whatever parameters it wants, Shouldn't it?
Project, | ||
] | ||
], | ||
[Any], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure this is the correct typing though.
The paging shouldn't care what the validation function it gets; Each validation function should define whatever parameters it wants, Shouldn't it?
…Ocean into PORT-fix-gitlab-exporter
Description
file://
is configured inport.yml
Type of change
Please leave one option from the following and delete the rest: