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(cli): assets are optional #3008

Merged
merged 1 commit into from
Nov 15, 2023
Merged

fix(cli): assets are optional #3008

merged 1 commit into from
Nov 15, 2023

Conversation

blacha
Copy link
Member

@blacha blacha commented Nov 15, 2023

Motivation

Assests are optional and should not be required to make a configuration bundle

Modification

ensure the assets are set as optional remove the required flag from the bundler cli

Checklist

If not applicable, provide explanation of why.

  • Tests updated
  • Docs updated
  • Issue linked in Title

@blacha blacha requested a review from a team as a code owner November 15, 2023 00:09
@Wentao-Kuang Wentao-Kuang added this pull request to the merge queue Nov 15, 2023
Merged via the queue into master with commit 5ea056b Nov 15, 2023
10 checks passed
@Wentao-Kuang Wentao-Kuang deleted the fix/cli-assets-opional branch November 15, 2023 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants