feat(superdeck): add watchForChanges option to DeckOptions #30
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add watchForChanges field to DeckOptions (defaults to false) - DeckControllerBuilder now respects watchForChanges before starting CLI watcher - Demo explicitly enables watchForChanges: true for development This allows consumers to disable the automatic CLI watcher process that monitors slides.md for changes.
|
To view this pull requests documentation preview, visit the following URL: Documentation is deployed and generated using docs.page. |
- Fix deprecated color.value usage in style_schemas_test.dart - Minor comment cleanup in test file - Regenerated raw_slide_schema.g.dart
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds a
watchForChangesoption toDeckOptionsthat controls whether the CLI watcher process is started automatically.Changes
DeckOptions: AddedwatchForChangesfield (defaults tofalse)DeckControllerBuilder: Now checkswatchForChangesbefore starting the CLI watcherwatchForChanges: trueto enable file watching during developmentBehavior
watchForChangesfalse(default)trueWhy
This allows consumers to disable the automatic CLI watcher process. By defaulting to
false, apps don't unexpectedly spawn a subprocess. The demo explicitly opts-in for development workflow.Testing
Pull Request opened by Augment Code with guidance from the PR author