-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat: add bruno int test collection #546
feat: add bruno int test collection #546
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.
Thanks a lot! Works great. I checked both, the GUI & the CLI usage and it works fine!
Had a few minor findings.
I already checked the license for the CLI and filed one IP check: https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/15955
Please handle comments. I'll then file a request at the EF.
...01-MAD/01_02-Operational Data/Supplier/Stock/Get Semiconductor Product Stocks -Supplier-.bru
Outdated
Show resolved
Hide resolved
...01-MAD/01_02-Operational Data/Supplier/Stock/Get Semiconductor Product Stocks -Supplier-.bru
Outdated
Show resolved
Hide resolved
local/bruno/puris-integration-test/environments/puris integration test env.bru
Outdated
Show resolved
Hide resolved
local/bruno/puris-integration-test/environments/puris integration test env.bru
Show resolved
Hide resolved
...puris-integration-test/Test_01-MAD/01_01-Init/Customer/MAD/Get Material Partner Mappings.bru
Outdated
Show resolved
Hide resolved
...ation-test/Test_01-MAD/01_01-Init/Supplier/MAD/Add New Site to Supplier Partner -MySelf-.bru
Outdated
Show resolved
Hide resolved
...puris-integration-test/Test_01-MAD/01_01-Init/Supplier/MAD/Get Material Partner Mappings.bru
Outdated
Show resolved
Hide resolved
...t/Test_01-MAD/01_02-Operational Data/Supplier/Delivery/Get Delivery 5 days Semiconductor.bru
Outdated
Show resolved
Hide resolved
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.
LGTM! Thanks a lot.
Regarding Trufflehog i raised PR#555.
As we got plenty lines of code I raised an EF IP issue. I will approve and merge as soon as we got the response :)
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.
IP check succeeded. Thanks a lot!
Description
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review: