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.

Sign up for GitHub

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

Jump to bottom

LPS-91390 SF #68946

Closed
wants to merge 2 commits into from
Closed

LPS-91390 SF #68946

wants to merge 2 commits into from

Conversation

pavel-savinov
Copy link

No description provided.

jcampoy and others added 2 commits March 1, 2019 13:08
@jcampoy
LPS-91390 Use logic for extracting the companyId from the request bec…
0750120 
…ause depending on the executed filters it could be not initialized as request attribute
@pavel-savinov
LPS-91390 SF
d0b1f21
@pavel-savinov pavel-savinov mentioned this pull request Mar 5, 2019
Closed
@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 35 seconds 706 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 09e4d5e1f9bf7afaea62bbe66cee2ce29d295cf9

Sender Branch:

Branch Name: pr-317
Branch GIT ID: d0b1f213d47b1b0b6d9faa349fc690acc9f7ab81

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

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 62 out of 81 jobs passed in 2 hours 49 minutes 39 seconds 281 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 09e4d5e1f9bf7afaea62bbe66cee2ce29d295cf9

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: bb05b18168fd1ff84bb232f565de282ff8c473b8

62 out of 81 jobs PASSED

19 Failed Jobs:

62 Successful Jobs:
For more details click here.

Failures unique to this pull:

  6. ...

@brianchandotcom
Copy link
Owner

Merged. Thx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci:test:relevant - failure ci:test:sf - success
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pavel-savinov @liferay-continuous-integration @brianchandotcom @jcampoy