docs: comprehensive code documentation for contributor onboarding#712
docs: comprehensive code documentation for contributor onboarding#712nameershah wants to merge 3 commits into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@nameershah I have no idea, sorry 😃 |
Added a default execution handler for Steps without a registered pipeline, including logging and error handling. Signed-off-by: Muhammad Nameer Shah <190166225+nameershah@users.noreply.github.com> Signed-off-by: Muhammad Nameer Shah <nameershah@users.noreply.github.com>
Added detailed docstring for the main entry point of the Kale KFServing Transformer server, including usage and argument descriptions. Signed-off-by: Muhammad Nameer Shah <190166225+nameershah@users.noreply.github.com> Signed-off-by: Muhammad Nameer Shah <nameershah@users.noreply.github.com>
Signed-off-by: Muhammad Nameer Shah <smns3960@gmail.com> Signed-off-by: Muhammad Nameer Shah <nameershah@users.noreply.github.com>
nameershah
force-pushed
the
docs/comprehensive-fix
branch
from
bc919b9 to
8bc49c7
Compare
|
Hi @ada333, I've fixed the DCO by signing off all commits. Could you re-approve when you get a chance? Thank you! |
|
Hello @nameershah Would it be possible to check the CI errors? Please try to build it locally to see if you have any error. Thanks! |
4 participants
Consolidating marshalling, step, and transformer documentation into a single signed-off PR.