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

convert should read the config, and respect its output option #838

Closed
Fil opened this issue Feb 16, 2024 · 1 comment · Fixed by #1579
Closed

convert should read the config, and respect its output option #838

Fil opened this issue Feb 16, 2024 · 1 comment · Fixed by #1579
Labels
enhancement New feature or request

Comments

@Fil
Copy link
Contributor

Fil commented Feb 16, 2024

#770 defaults --output to . It would be nicer to default to docs/ (or the root if specified in the config), so users can start preview/build faster without having to figure out this option.

@Fil Fil added the enhancement New feature or request label Feb 16, 2024
@mbostock mbostock changed the title convert should default --output to docs convert should read the config, and respect its output option Feb 17, 2024
@mbostock
Copy link
Member

Retitled and redesigned: I think it should take the --config option (which defaults to observablehq.config.{ts,js} in the current working directory) and then respect the output option in the config, which defaults to docs. So if you have a different output set, --output will default to that. And maybe we don’t bother reading the config if you specify --output explicitly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants