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-89549 Prefer using getElementById #67671

Closed
wants to merge 3 commits into from
Closed

LPS-89549 Prefer using getElementById #67671

wants to merge 3 commits into from

Conversation

p2kmgcl
Copy link

@p2kmgcl p2kmgcl commented Feb 6, 2019

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering "ci:test:sf" and "ci:test:relevant" for this pull to run Source Formatter and relevant tests.

Comment "ci:test" to run the full PR Tester for this pull.

@liferay-continuous-integration
Copy link
Collaborator

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 6e9ff2489f58b5620382ae1a3aba895f8701f43e

Sender Branch:

Branch Name: pr-247
Branch GIT ID: 6f1900c80a3b46edbfb86beb8bec020d42446495

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

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 20 out of 22 jobs passed in 1 hour 30 minutes 41 seconds 251 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 6e9ff2489f58b5620382ae1a3aba895f8701f43e

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 440a521f80aba216281e29410e63788cc0c97766

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

This pull contains no unique failures.


Failures in common with acceptance upstream results at 6e9ff24:

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 84faa89...fadd9fb

@p2kmgcl p2kmgcl deleted the pr-247 branch May 16, 2019 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants