-
Notifications
You must be signed in to change notification settings - Fork 5
Add Typespec API definitions #537
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
Conversation
Co-authored-by: Phil Ewels <[email protected]>
Tried this and got many errors.
|
Signed-off-by: munishchouhan <[email protected]>
Signed-off-by: munishchouhan <[email protected]>
@pditommaso please run these commands, I have updated the docs too
|
Much better, only to errors 😄
|
@pditommaso please |
Good ✌️ |
To close this, think we need to add a Github action that checks the model is correct, likely running |
ok i will add that |
Signed-off-by: munishchouhan <[email protected]>
getting rate limit reached error from docker hub, will try and run tests tomorrow
|
@pditommaso added validation step in CI |
Signed-off-by: munishchouhan <[email protected]>
Signed-off-by: munishchouhan <[email protected]>
This PR will add documentation based on typespec