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

feat: fal compressed file #17

Closed
wants to merge 27 commits into from

Conversation

badayvedat
Copy link
Contributor

@badayvedat badayvedat commented Dec 26, 2023

this branch is built on top of #16, so #16 needs to be reviewed before this PR.

badayvedat and others added 2 commits December 26, 2023 00:06
Co-authored-by: Daniel Rochetti <[email protected]>
Co-authored-by: Daniel Rochetti <[email protected]>
@badayvedat badayvedat changed the base branch from main to refactor-fal-toolkit-file January 10, 2024 15:35
badayvedat and others added 15 commits January 10, 2024 18:38
* wip: feat: `fal.App` for multiple endpoints

* fix: build the metadata

* add tests
…n unauth exception (#32)

* chore: print hint of exception if available

* add reference to fal keys in unauthenticated exception
[create-pull-request] automated change

Co-authored-by: chamini2 <[email protected]>
* ci: avoid committing datadog keys

* ci: fix remove all git changes but remember built version
[create-pull-request] automated change

Co-authored-by: chamini2 <[email protected]>
* chore: update openapi client

* chore: update pre-commit to ignore openapi-fal-rest client
Comment on lines +88 to +94

# set in GITHUB_ENV
BUILT_VERSION=$(poetry version -s)
echo "BUILT_VERSION=$BUILT_VERSION"
echo "BUILT_VERSION=$BUILT_VERSION" >> $GITHUB_ENV

echo "version=$BUILT_VERSION" >> "$GITHUB_OUTPUT"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we editing the release files?

@badayvedat badayvedat closed this Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants