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] add jinja2 #104

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AlekseyKorshuk
Copy link
Contributor

@AlekseyKorshuk AlekseyKorshuk commented Oct 28, 2024

Adds Jinja2 to the requirements to improve the prompting experience

Note: jinja2 is already included in transformers, but it is better to enforce its existence

Adds Jinja2 to the requirements to improve the prompting experience
@jzleibo
Copy link
Collaborator

jzleibo commented Oct 30, 2024

This looks fine to me, I think you just have to update your branch to the latest commit, then it will let me click a button to start the process to merge it.

@AlekseyKorshuk
Copy link
Contributor Author

@jzleibo Done

@jzleibo jzleibo self-assigned this Oct 30, 2024
@jzleibo jzleibo requested a review from vezhnick October 30, 2024 13:25
copybara-service bot pushed a commit that referenced this pull request Oct 30, 2024
--
f428077 by Aleksey Korshuk <[email protected]>:

[feat] add jinja2

Adds Jinja2 to the requirements to improve the prompting experience

COPYBARA_INTEGRATE_REVIEW=#104 from AlekseyKorshuk:patch-2 da077e8
PiperOrigin-RevId: 691402699
Change-Id: Id40a659801c44378812a2168b1487af346767850
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.

3 participants