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-73075 Optimize AssetLinksTag to avoid dispatching unless there ar… #49624

Closed
wants to merge 1 commit into from

Conversation

shuyangzhou
Copy link

…e AssetLinks to render.

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

The pull request tester is still running.

Please wait until you get the final report before running 'ci:retest'.

See this link to check on the status of your test:

@shuyangzhou


However, the pull request was closed.

The pull request was closed because the following critical batches had failed:

For information as to why we automatically close out certain pull requests see this article.

*This pull will no longer automatically close if this comment is available. If you believe this is a mistake please re-open this pull by entering the following command as a comment.

ci:reopen

Critical Failure Details:

@shuyangzhou
Copy link
Author

ci:reopen

@liferay-continuous-integration
Copy link
Collaborator

Some tests FAILED.

Build Time: 2 hours 14 minutes 3 seconds 490 ms

Base Branch:

Branch Name: master
Branch GIT ID: 66bd9867e86edfebf8930a4d0799ff1fe2522736

Job Summary:

For more details click here.

Failed Jobs:

  1. test-portal-acceptance-pullrequest(master)
    Job Results:

    100 Jobs Passed.
    9 Jobs Failed.

    Downstream jobs FAILED.
  2. ...
For upstream results, click here.

@brianchandotcom
Copy link
Owner

@shuyangzhou please backport. Thx.

@brianchandotcom
Copy link
Owner

@shuyangzhou nvm, for this one, I was able to cherry pick cleanly.

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.

3 participants