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

fix(compiler): no generate custom output #5951

Merged

Conversation

Sukaato
Copy link
Contributor

@Sukaato Sukaato commented Aug 19, 2024

Do not generate custom output target on test

fixes: #5950

What is the current behavior?

GitHub Issue Number: #5950

What is the new behavior?

No generating the custom output target on test build

Documentation

Does this introduce a breaking change?

  • Yes
  • No

Testing

Manual testing

Other information

@Sukaato Sukaato requested a review from a team as a code owner August 19, 2024 19:03
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

Thanks for raising the PR. This looks good to me but I would like to check in with the team to discuss the new behavior.

Copy link
Member

@tanner-reits tanner-reits left a comment

Choose a reason for hiding this comment

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

I think this is fine. We do something similar for documentation generation by setting a config.buildDocs to false in the testing task, but this does essentially the same thing.

@christian-bromann christian-bromann added this pull request to the merge queue Aug 26, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 26, 2024
Do not generate custom output target on test

fixes: ionic-team#5950
@Sukaato Sukaato force-pushed the fix/do-not-generated-custom-on-test branch from 811dfe2 to db661ab Compare August 27, 2024 11:42
@Sukaato
Copy link
Contributor Author

Sukaato commented Aug 28, 2024

CI / Unit Tests / (windows-latest.22) failed, but it is not related to my commit i guess.
any advice to resolve the failed check ?

@christian-bromann
Copy link
Member

any advice to resolve the failed check ?

Unfortunately we haven't had the chance to identify the root cause of these flakes in our pipelines. I just re-ran the failing test and will go ahead and merge once passing.

@christian-bromann christian-bromann added this pull request to the merge queue Aug 30, 2024
Merged via the queue into ionic-team:main with commit 5cddfd9 Aug 30, 2024
88 checks passed
@Sukaato Sukaato deleted the fix/do-not-generated-custom-on-test branch August 30, 2024 21:47
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.

bug: stencil test with e2e trigger documentation build
3 participants