Skip to content

test: complex e2e test case#215

Merged
Bobgy merged 1 commit intogoogle:masterfrom
inteon:add_complex_e2e
Jul 10, 2023
Merged

test: complex e2e test case#215
Bobgy merged 1 commit intogoogle:masterfrom
inteon:add_complex_e2e

Conversation

@inteon
Copy link
Copy Markdown
Contributor

@inteon inteon commented Jul 8, 2023

This PR adds a more complex e2e test (a module with lots of dependencies).

@inteon inteon mentioned this pull request Jul 8, 2023
@inteon inteon force-pushed the add_complex_e2e branch from f718896 to 80d3bd7 Compare July 10, 2023 07:16
Copy link
Copy Markdown
Collaborator

@Bobgy Bobgy left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM

github.com/Azure/go-autorest/autorest,https://github.com/Azure/go-autorest/blob/autorest/v0.11.29/autorest/LICENSE,Apache-2.0
github.com/Azure/go-autorest/autorest/adal,https://github.com/Azure/go-autorest/blob/autorest/adal/v0.9.23/autorest/adal/LICENSE,Apache-2.0
github.com/Azure/go-autorest/autorest/date,https://github.com/Azure/go-autorest/blob/autorest/date/v0.3.0/autorest/date/LICENSE,Apache-2.0
github.com/Azure/go-autorest/autorest/to,https://github.com/Azure/go-autorest/blob/autorest/to/v0.4.0/autorest/to/LICENSE,Apache-2.0
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The link is incorrect, but it's a known issue, license path for sub module can have problems.

@Bobgy Bobgy changed the title Add complex e2e test: complex e2e test case Jul 10, 2023
@Bobgy Bobgy merged commit 150520f into google:master Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants