Conversation
2 tasks
Member
|
👁️🗨️ I notice the tests aren't starting for this PR. IIRC this is a limitation of app created PRs 🤖 I'm checking out options of starting tests with We also might ignore the required checks instead since tests on the |
62b8efc to
f617484
Compare
f617484 to
305b030
Compare
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
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.
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or setup this action to publish automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@slack/cli-hooks@1.3.0
Minor Changes
0abdc91: feat(cli-hooks): add default app and manifest watch config
This package now provides default watch configurations for automatic file watching during
slack run. The CLI will restart your app server when source files change and reinstall your app when the manifest changes.Requirements: These features require Slack CLI v3.12.0+ with file watching support.
Default Configuration
The following watch settings are provided automatically when using this package:
{ "config": { "watch": { "app": { "filter-regex": "\\.js$", "paths": ["."] }, "manifest": { "paths": ["manifest.json"] } } } }Note: Manifest watching requires a local manifest source in your
.slack/config.jsonfile. Remote manifests will not be updated on file changes.{ "manifest": { "source": "local" } }Custom Configurations
You can override these defaults in your
.slack/hooks.jsonfile to reduce the paths searched or change the file patterns. Read Watch Configurations for more options.TypeScript Development
TypeScript developers should run
tsc --watchin a separate terminal during development. This compiles.tsfiles to.json changes, and the default watch configuration will detect changes to the compileddist/*.jsfiles and restart the app server. This approach works best with the default settings.Patch Changes
8962739: fix(cli-hooks): stop app process if the start hook exits
Fixes a CLI issue where daemon app processes were spawned if the CLI was exited without being interrupted.