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
feat: emit schedule
property into manifest
#768
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I thought it'd be a good idea to formally define the format of the manifest file, especially as other systems will start to consume it, so I've created #770.
schedule
property into manifestschedule
property into manifest
@netlify-team-account-1 let's hold off on merging this until #760 is released (which is waiting for #769). |
Co-authored-by: Eduardo Bouças <mail@eduardoboucas.com>
⏱ Benchmark resultsComparing with 3c982bd largeDepsEsbuild: 9.3s⬆️ 16.69% increase vs. 3c982bd
Legend
largeDepsZisi: 1m 18.4s⬆️ 18.92% increase vs. 3c982bd
Legend
|
tests/main.js
Outdated
@@ -2147,7 +2147,7 @@ if (platform !== 'win32') { | |||
}) | |||
} | |||
|
|||
test.only('Creates a manifest file with the list of created functions if the `manifest` property is supplied', async (t) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eduardoboucas should we have some lint rule against leaving those in? 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, definitely!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✨
- Summary
Adds a new config field called
schedule
, which is emitted into the functions manifest.TODO: mention in documentation.
- Test plan
added unit test.
- A picture of a cute animal (not mandatory but encouraged)