You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@amoro.apache.org by "zhoujinsong (via GitHub)" <gi...@apache.org> on 2024/04/03 01:39:55 UTC

Re: [PR] [Hotfix] Add frontend plugin to build frontend automatically [incubator-amoro]

zhoujinsong commented on PR #2670:
URL: https://github.com/apache/incubator-amoro/pull/2670#issuecomment-2033385499

   > @zhoujinsong Thanks for your contribution. Do we not need the `frontend` profile? Would it be better to keep this and change the default value to true?
   
   It seems necessary to build the frontend now, otherwise the unit tests for AMS will fail, and the packaged AMS won't run properly. Therefore, I don't think there's any need to retain this profile anymore.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@amoro.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org