-
Notifications
You must be signed in to change notification settings - Fork 36
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
Cleanup and document engine-sample.input
#580
Cleanup and document engine-sample.input
#580
Conversation
|
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #580 +/- ##
=======================================
Coverage 29.71% 29.71%
=======================================
Files 86 86
Lines 6158 6158
=======================================
Hits 1830 1830
Misses 4328 4328 ☔ View full report in Codecov by Sentry. |
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.
This sample is overcomplicated (my bad), some suggestions to simplify it:
engine-sample.input
engine-sample.input
engine-sample.input
5a414b9
to
c97b77e
Compare
c97b77e
to
01cd6e1
Compare
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.
I think you should mention in the docs thar Input
is a resource and that the showcase functions you're showing are not systems, and that there's a system passing the input resource there.
Maybe you should show that system or part of it too, just to make it clear where stuff is coming from.
01cd6e1
to
6273cf5
Compare
No description provided.