-
Notifications
You must be signed in to change notification settings - Fork 51
Setup Codeflash Github Actions to optimize all future code #596
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
base: main
Are you sure you want to change the base?
Conversation
nltk env var bug
|
||
- name: Codeflash | ||
run: | | ||
uv sync --all-groups --all-extras --upgrade |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this uv sync line may cause a pretty big install. lets keep an eye on it
uv sync --all-groups --all-extras --upgrade | ||
uv pip install unstructured | ||
uv pip install codeflash | ||
uv run python -m nltk.downloader -d $NLTK_DATA punkt_tab averaged_perceptron_tagger_eng |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assuming we need nltk.downloader as a requirement for unstructured?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
we may need the version and changelog.. we will find out
codeflash.yml
file which will run on every new Pull Request that modifies the source code forunstructured-ingest
directory.Actions Required to make this work-
CODEFLASH_API_KEY
after signing up to Codeflash from our website. The onboarding will ask you to create an API Key and show instructions on how to save the api key on your repo secrets.Then, after this PR is merged in it will start generating new optimizations 🎉