Skip to content

Conversation

@ozabluda
Copy link
Contributor

@ozabluda ozabluda commented Jul 6, 2023

Rename prompt_template => _DEFAULT_GRAPH_QA_TEMPLATE to make consistent with the rest of the file.

Rename prompt_template => _DEFAULT_GRAPH_QA_TEMPLATE to make consistent with the rest of the file.
@vercel
Copy link

vercel bot commented Jul 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Jul 6, 2023 6:36pm

@dosubot dosubot bot added the auto:nit label Jul 6, 2023
Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

thanks - want to rename PROMPT as well?

@hwchase17 hwchase17 added the lgtm label Jul 6, 2023
@ozabluda ozabluda changed the title Rename prompt_template => _DEFAULT_GRAPH_QA_TEMPLATE to make consistent with the rest of the file Rename prompt_template => _DEFAULT_GRAPH_QA_TEMPLATE and PROMPT => _GRAPH_QA_PROMPT to make consistent with the rest of the files Jul 6, 2023
@ozabluda ozabluda changed the title Rename prompt_template => _DEFAULT_GRAPH_QA_TEMPLATE and PROMPT => _GRAPH_QA_PROMPT to make consistent with the rest of the files Rename prompt_template => _DEFAULT_GRAPH_QA_TEMPLATE and PROMPT => GRAPH_QA_PROMPT to make consistent with the rest of the files Jul 6, 2023
@ozabluda
Copy link
Contributor Author

ozabluda commented Jul 6, 2023

OK, renamed PROMPT as well

@baskaryan baskaryan merged commit 9954eff into langchain-ai:master Jul 7, 2023
@ozabluda ozabluda deleted the patch-1 branch July 7, 2023 06:30
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