mirror of
https://github.com/hwchase17/langchain.git
synced 2025-07-19 19:11:33 +00:00
infra: PR template nits (#17752)
This commit is contained in:
parent
ad285ca15c
commit
a9d3c100a2
17
.github/PULL_REQUEST_TEMPLATE.md
vendored
17
.github/PULL_REQUEST_TEMPLATE.md
vendored
@ -1,19 +1,24 @@
|
|||||||
Thank you for contributing to LangChain!
|
Thank you for contributing to LangChain!
|
||||||
|
|
||||||
Checklist:
|
- [ ] **PR title**: "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.
|
||||||
- [ ] 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.
|
|
||||||
- Example: "community: add foobar LLM"
|
- Example: "community: add foobar LLM"
|
||||||
- [ ] PR message: **Delete this entire template message** and replace it with the following bulleted list
|
|
||||||
|
|
||||||
|
- [ ] **PR message**: ***Delete this entire checklist*** and replace with
|
||||||
- **Description:** a description of the change
|
- **Description:** a description of the change
|
||||||
- **Issue:** the issue # it fixes, if applicable
|
- **Issue:** the issue # it fixes, if applicable
|
||||||
- **Dependencies:** any dependencies required for this change
|
- **Dependencies:** any dependencies required for this change
|
||||||
- **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out!
|
- **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/
|
|
||||||
- [ ] Add tests and docs: If you're adding a new integration, please include
|
|
||||||
|
- [ ] **Add tests and docs**: 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.
|
||||||
|
|
||||||
|
|
||||||
|
- [ ] **Lint and test**: Run `make format`, `make lint` and `make test` from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/
|
||||||
|
|
||||||
Additional guidelines:
|
Additional guidelines:
|
||||||
- Make sure optional dependencies are imported within a function.
|
- 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.
|
- Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
|
||||||
|
Loading…
Reference in New Issue
Block a user