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

fix(jit): Input arguments required error #2756

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

karatakis
Copy link
Collaborator

@karatakis karatakis commented Aug 26, 2024

Summary:
This PR will enable Tailcall JIT implementation to validate the Input arguments if their fields meet the schema requirements. When an argument does not meet the requirements, an error is thrown.

Fixes:

Build & Testing:

  • I ran cargo test successfully.
  • I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

  • I have added relevant unit & integration tests.
  • I have updated the documentation accordingly.
  • I have performed a self-review of my code.
  • PR follows the naming convention of <type>(<optional scope>): <title>

@github-actions github-actions bot added the type: fix Iterations on existing features or infrastructure. label Aug 26, 2024
@karatakis karatakis changed the title fix(jit):Arguments required error fix(jit):Input arguments required error Aug 27, 2024
Copy link

github-actions bot commented Sep 1, 2024

Action required: PR inactive for 5 days.
Status update or closure in 10 days.

@github-actions github-actions bot added the state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. label Sep 1, 2024
Copy link

PR closed after 10 days of inactivity.

@github-actions github-actions bot closed this Sep 11, 2024
@tusharmath tusharmath reopened this Sep 11, 2024
Copy link

codecov bot commented Sep 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.19%. Comparing base (56bdf27) to head (2f595f9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2756      +/-   ##
==========================================
+ Coverage   87.13%   87.19%   +0.05%     
==========================================
  Files         254      254              
  Lines       25144    25170      +26     
==========================================
+ Hits        21910    21946      +36     
+ Misses       3234     3224      -10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot removed the state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. label Sep 11, 2024
@karatakis karatakis changed the title fix(jit):Input arguments required error fix(jit): Input arguments required error Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: fix Iterations on existing features or infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug If I need to inlining UserData fails in jit
2 participants