Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

OPIK-985: Regenerate OpenAi code #1387

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

thiagohora
Copy link
Contributor

@thiagohora thiagohora commented Feb 26, 2025

Details

Regenerate OpenAi code and docs

@thiagohora thiagohora self-assigned this Feb 26, 2025
Copy link
Contributor

@thiagohora thiagohora force-pushed the thiagohora/OPIK-985_regenerate_openapi_code branch from 9693152 to a79c225 Compare February 26, 2025 11:02
@thiagohora thiagohora marked this pull request as ready for review February 26, 2025 11:03
@thiagohora thiagohora requested review from a team as code owners February 26, 2025 11:03
Copy link
Contributor

@thiagohora
Copy link
Contributor Author

@ferc can I ignore this linter check?

@ferc
Copy link
Contributor

ferc commented Feb 26, 2025

@thiagohora yes, in fact the auto-generated folder is ignored for the linter here: https://github.com/comet-ml/opik/blob/main/sdks/typescript/eslint.config.js#L7 but somehow in this CI run it doesn't take that into account

Feel free to merge as is, I'll investigate why is not working as it should

@thiagohora thiagohora merged commit 5369ef6 into main Feb 26, 2025
48 of 49 checks passed
@thiagohora thiagohora deleted the thiagohora/OPIK-985_regenerate_openapi_code branch February 26, 2025 11:38
@ferc
Copy link
Contributor

ferc commented Feb 26, 2025

Okay, leaving this comment for future times, it wasn't the linter, it was the TypeScript types, the new auto-generated code has broken types = build does not work

I'll try to figure it out how to solve in another PR but for future times let's double-check if it's linter (which should be excluded) or types issues

FYI @aadereiko

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants