-
Notifications
You must be signed in to change notification settings - Fork 41
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
chore: only run component build from preview middleware #2577
Conversation
test
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change is straightforward
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
Quality Gate passedIssues Measures |
* origin/main: (45 commits) chore: apply latest changesets Removes console log statement, adds some missing jsdoc (#2576) chore: only run component build from preview middleware (#2577) chore: apply latest changesets fix(ui-components): UITranslationInput. Screen readers read the resolved translation tooltip/title of UITranslationInput as part of the screen reader speech (#2568) fix: manifest changes not applied after reload (#2569) chore: apply latest changesets tbi(odata-service-inquirer): Move `ErrorHandler` and `getCFAbapInstanceChoices` to allow re-use (#2557) chore: apply latest changesets fix: adaptation editor crash (#2562) chore: apply latest changesets chore: update dependencies (#2558) chore: apply latest changesets fix(preview-middleware): typo (#2560) chore: apply latest changesets chore: enable quick actions (#2534) chore: apply latest changesets feat(cpe): display and write app descriptor changes for v4 (#2458) chore: apply latest changesets feat(@sap-ux/odata service inquirer): Adds system selection prompting (#2374) ...
* origin/main: (54 commits) chore: apply latest changesets fix(cf-deploy-config-writer): cleanup based on local testing when using headless (#2546) chore: apply latest changesets Removes console log statement, adds some missing jsdoc (#2576) chore: only run component build from preview middleware (#2577) chore: apply latest changesets fix(ui-components): UITranslationInput. Screen readers read the resolved translation tooltip/title of UITranslationInput as part of the screen reader speech (#2568) fix: manifest changes not applied after reload (#2569) chore: apply latest changesets tbi(odata-service-inquirer): Move `ErrorHandler` and `getCFAbapInstanceChoices` to allow re-use (#2557) chore: apply latest changesets fix: adaptation editor crash (#2562) chore: apply latest changesets chore: update dependencies (#2558) chore: apply latest changesets fix(preview-middleware): typo (#2560) chore: apply latest changesets chore: enable quick actions (#2534) chore: apply latest changesets feat(cpe): display and write app descriptor changes for v4 (#2458) ...
After 8906662 build is failing because references from
@sap-ux-private/control-property-editor-common
could not be resolved. We do not actually need to run type checking frompreview-middleware
as we only need to copy the artefacts over, so we can run only component build and relay on type checking being done bypreview-middleware-client
build.