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

feat(risedev): add build dashboard config #1086

Merged
merged 3 commits into from
Mar 20, 2022

Conversation

skyzh
Copy link
Contributor

@skyzh skyzh commented Mar 20, 2022

Signed-off-by: Alex Chi [email protected]

What's changed and what's your intention?

RiseDev will now export ENABLE_BUILD_DASHBOARD_V2 env variable if we want to build dashboard v2. Dashboard developer will need to update Makefile.toml to build or git checkout the dashboard artifact. I've supported it along with this PR.

Also, the env file will have more information on each option.

Checklist

  • I have written necessary docs and comments
  • I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Copy link
Contributor

@cloudcarver cloudcarver left a comment

Choose a reason for hiding this comment

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

LGSTM!

@skyzh skyzh mentioned this pull request Mar 20, 2022
2 tasks
Signed-off-by: Alex Chi <[email protected]>
@skyzh skyzh force-pushed the skyzh/add-dashboard-risedev-config branch from 44f15c9 to cae9f6c Compare March 20, 2022 07:35
@skyzh skyzh enabled auto-merge (squash) March 20, 2022 07:36
@skyzh skyzh merged commit 9ee35b3 into main Mar 20, 2022
@skyzh skyzh deleted the skyzh/add-dashboard-risedev-config branch March 20, 2022 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants