-
Notifications
You must be signed in to change notification settings - Fork 16
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
d6e1ee3
commit 604b662
Showing
1 changed file
with
47 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,47 @@ | ||
### Closes https://github.com/elastic/crawler/issues/### | ||
|
||
<!--Provide a general description of the code changes in your pull request. | ||
If the change relates to a specific issue, include the link at the top. | ||
If this is an ad-hoc/trivial change and does not have a corresponding | ||
issue, please describe your changes in enough details, so that reviewers | ||
and other team members can understand the reasoning behind the pull request.--> | ||
|
||
### Checklists | ||
|
||
<!--You can remove unrelated items from checklists below and/or add new | ||
items that may help during the review.--> | ||
|
||
#### Pre-Review Checklist | ||
- [ ] This PR does NOT contain credentials of any kind, such as API keys or username/passwords (double check `crawler.yml.example` and `elasticsearch.yml.example`) | ||
- [ ] This PR has a meaningful title | ||
- [ ] This PR links to all relevant GitHub issues that it fixes or partially addresses | ||
- If there is no GitHub issue, please create it. Each PR should have a link to an issue | ||
- [ ] this PR has a thorough description | ||
- [ ] Covered the changes with automated tests | ||
- [ ] Tested the changes locally | ||
- [ ] Added a label for each target release version (example: `v0.1.0`) | ||
- [ ] Considered corresponding documentation changes | ||
- [ ] Contributed any configuration settings changes to the configuration reference | ||
|
||
#### Changes Requiring Extra Attention | ||
|
||
<!--Please call out any changes that require special attention from the | ||
reviewers and/or increase the risk to availability or security of the | ||
system after deployment. Remove the ones that don't apply.--> | ||
|
||
- [ ] Security-related changes (encryption, TLS, SSRF, etc) | ||
- [ ] New external service dependencies added. | ||
|
||
### Related Pull Requests | ||
|
||
<!--List any relevant PRs here or remove the section if this is a standalone PR. | ||
* https://github.com/elastic/.../pull/123--> | ||
|
||
### Release Note | ||
|
||
<!--If you think this enhancement/fix should be included in the release notes, | ||
please write a concise user-facing description of the change here. | ||
You should also label the PR with `release_note` so the release notes | ||
author(s) can easily look it up.--> |