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.

Sign up for GitHub

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

Jump to bottom

trace-pkg, serverless-jetpack: Allow globs on more options. #15

Open
4 tasks
ryan-roemer opened this issue Sep 14, 2022 · 0 comments
Open
4 tasks

trace-pkg, serverless-jetpack: Allow globs on more options. #15

ryan-roemer opened this issue Sep 14, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@ryan-roemer
Copy link
Member

ryan-roemer commented Sep 14, 2022 •
edited
Loading

Using trace-pkg as an example, these options accept globs already:

  • include
  • trace

These options accept path prefixes (so theoretically don't need globs)

  • ignores

These are options we don't yet provide glob support for but should check / research / add notes:

  • dynamic.resolutions
    • app sources (start with ./)
    • dependencies
  • allowMissing

Previous related issues:
@ryan-roemer ryan-roemer added the enhancement New feature or request label Sep 14, 2022
@ryan-roemer ryan-roemer mentioned this issue Sep 16, 2022
Merged
ryan-roemer added a commit that referenced this issue Sep 16, 2022
@ryan-roemer
Chore: Update README and fix notes about glob support. (#16)
bc56e5b 
Part of #15 

- Confirms that `dynamic.resolutions` does _not_ support globs and updates README
- Adds a helper README to trace-pkg tests to explain what each type of test does
- Add some neato emojis
@ryan-roemer ryan-roemer mentioned this issue Dec 16, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ryan-roemer