-
Notifications
You must be signed in to change notification settings - Fork 14
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: adding readme suffixes to the manifest #154
Conversation
templates/consoleReadmeSuffix.md
Outdated
To learn more about Apify and Actors, take a look at the following resources: | ||
|
||
- [Apify SDK for JavaScript documentation](https://docs.apify.com/sdk/js) | ||
- [Apify SDK for Python documentation](https://docs.apify.com/sdk/python) |
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.
Added the Python documentation here in case the template is python-based. This could be dynamic somehow as well but don't think it is worth the effort
Co-authored-by: Ondra Urban <23726914+mnmkng@users.noreply.github.com>
Co-authored-by: Jan Bárta <45016873+jbartadev@users.noreply.github.com>
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.
Looks good, just small nitpicks
Co-authored-by: Jan Bárta <45016873+jbartadev@users.noreply.github.com>
FYI you can use node-fetch instead https as it's already in package-lock. |
Yeah but only in dev dependencies, this package has no production dependencies |
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.
Thanks, much more helpful now 👍
closes apify/apify-web#2708