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.

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

Add INPUT schema checks to "apify run" #66

Open
jancurn opened this issue Oct 30, 2018 · 2 comments · May be fixed by #560
Open

Add INPUT schema checks to "apify run" #66

jancurn opened this issue Oct 30, 2018 · 2 comments · May be fixed by #560
Assignees
Labels
enhancement New feature or request. t-tooling Issues with this label are in the ownership of the tooling team.
Milestone

Comments

@jancurn
Copy link
Member

jancurn commented Oct 30, 2018

If actor has an INPUT schema, then "apify run" should validate the input against the schema, using the exactly same code as Apify cloud is doing that.

@jancurn jancurn added the enhancement New feature or request. label Oct 30, 2018
@jancurn
Copy link
Member Author

jancurn commented Oct 30, 2018

@mtrunkat is the schema validation code somewhere accessible?

@jancurn
Copy link
Member Author

jancurn commented Oct 30, 2018

Plus it should validate that the schema is valid. But who will validate that the validator of the schema is valid, eh? :)

@mtrunkat mtrunkat added the t-platform Issues with this label are in the ownership of the platform team. label Aug 1, 2022
@B4nan B4nan added t-tooling Issues with this label are in the ownership of the tooling team. and removed t-platform Issues with this label are in the ownership of the platform team. labels Mar 11, 2024
@B4nan B4nan added this to the 1.0 milestone Apr 23, 2024
@vladfrangu vladfrangu linked a pull request Apr 30, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request. t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants