mirror of
https://github.com/hwchase17/langchain.git
synced 2025-07-03 19:57:51 +00:00
infra: update pr template (#17437)
This commit is contained in:
parent
b48fa8b695
commit
37629516cd
34
.github/PULL_REQUEST_TEMPLATE.md
vendored
34
.github/PULL_REQUEST_TEMPLATE.md
vendored
@ -1,20 +1,24 @@
|
|||||||
<!-- Thank you for contributing to LangChain!
|
Thank you for contributing to LangChain!
|
||||||
|
|
||||||
Please title your PR "<package>: <description>", where <package> is whichever of langchain, community, core, experimental, etc. is being modified.
|
Checklist:
|
||||||
|
|
||||||
Replace this entire comment with:
|
- [ ] PR title: Please title your PR "package: description", where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..." for purely docs changes, "templates: ..." for template changes, "infra: ..." for CI changes.
|
||||||
- **Description:** a description of the change,
|
- Example: "community: add foobar LLM"
|
||||||
- **Issue:** the issue # it fixes if applicable,
|
- [ ] PR message: Delete this entire template and replace it with the following bulleted list
|
||||||
- **Dependencies:** any dependencies required for this change,
|
- **Description:** a description of the change
|
||||||
- **Twitter handle:** we announce bigger features on Twitter. If your PR gets announced, and you'd like a mention, we'll gladly shout you out!
|
- **Issue:** the issue # it fixes, if applicable
|
||||||
|
- **Dependencies:** any dependencies required for this change
|
||||||
Please make sure your PR is passing linting and testing before submitting. Run `make format`, `make lint` and `make test` from the root of the package you've modified to check this locally.
|
- **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out!
|
||||||
|
- [ ] Pass lint and test: Run `make format`, `make lint` and `make test` from the root of the package(s) you've modified to check that you're passing lint and testing. See contribution guidelines for more information on how to write/run tests, lint, etc: https://python.langchain.com/docs/contributing/
|
||||||
See contribution guidelines for more information on how to write/run tests, lint, etc: https://python.langchain.com/docs/contributing/
|
- [ ] Add tests and docs: If you're adding a new integration, please include
|
||||||
|
|
||||||
If you're adding a new integration, please include:
|
|
||||||
1. a test for the integration, preferably unit tests that do not rely on network access,
|
1. a test for the integration, preferably unit tests that do not rely on network access,
|
||||||
2. an example notebook showing its use. It lives in `docs/docs/integrations` directory.
|
2. an example notebook showing its use. It lives in `docs/docs/integrations` directory.
|
||||||
|
|
||||||
If no one reviews your PR within a few days, please @-mention one of @baskaryan, @eyurtsev, @hwchase17.
|
Additional guidelines:
|
||||||
-->
|
- Make sure optional dependencies are imported within a function.
|
||||||
|
- Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
|
||||||
|
- Most PRs should not touch more than one package.
|
||||||
|
- Changes should be backwards compatible.
|
||||||
|
- If you are adding something to community, do not re-import it in langchain.
|
||||||
|
|
||||||
|
If no one reviews your PR within a few days, please @-mention one of @baskaryan, @efriis, @eyurtsev, @hwchase17.
|
Loading…
Reference in New Issue
Block a user