Skip to content
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

LPS-112018 Remove <clay:button> and use hardcoded markup #87685

Conversation

liferay-continuous-integration
Copy link
Collaborator

Forwarded from: 4lejandrito#460

@4lejandrito

Original pull request comment:
@ambrinchaudhary, @boton, @dantewang, @GinsonRen

✔️ ci:test:stable - 17 out of 17 jobs passed

✔️ ci:test:relevant - 29 out of 29 jobs passed in 3 hours 23 seconds 115 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: e145c48e9115bd2ede749d802e49483b5341bf78

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: dedb2cf07c7105cbb78b2a4f981022f7a7a84946

ci:test:stable - 17 out of 17 jobs PASSED
17 Successful Jobs:
ci:test:relevant - 29 out of 29 jobs PASSED
29 Successful Jobs:
For more details click here.

✔️ ci:test:sf - 1 out of 1 jobs passed in 2 minutes 45 seconds 390 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: e145c48e9115bd2ede749d802e49483b5341bf78

Sender Branch:

Branch Name: pr-459
Branch GIT ID: bb9f21267fe40d5ad2d0834cd554d6cdeec174b9

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator Author

To conserve resources, the PR Tester does not automatically run for forwarded pull requests.

@4lejandrito
Copy link

Hi @brianchandotcom,,

This fixes a performance regression. We need to keep dd25b89 to avoid unnecessary calculations.

Thanks!

@brianchandotcom
Copy link
Owner

@4lejandrito slack me how bad the perf is? Thx.

@brianchandotcom
Copy link
Owner

Merged. Thx.

@liferay-continuous-integration liferay-continuous-integration deleted the ci-forward-pr-459-pr-460-sender-4lejandrito-ts-1587047559311 branch April 24, 2020 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants