Skip to content

chore: update issue and pr templates #796

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

Merged
merged 3 commits into from
Nov 15, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 26 additions & 2 deletions .github/ISSUE_TEMPLATE/bug_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,17 +25,27 @@ body:
label: A link to a reproduction repository
description: The fastest way to describe your bug is to provide a reproduction repository.
placeholder: https://github.com/project/link-to-your-reproduction-repo

- type: markdown
attributes:
value: |
Check out [CONTRIBUTING.md](https://github.com/netlify/netlify-plugin-nextjs/blob/main/CONTRIBUTING.md#what-is-a-reproducible-test-case) for advice on producing a reproduction repo.
- type: input
id: plugin-version
attributes:
label: Plugin version
description: The version of `@netlify/plugin-nextjs` that you are using. (This is in the build logs)
placeholder: x.x.x
validations:
required: true

- type: checkboxes
id: more-info
attributes:
label: More information about your build
options:
- label: I am building using the CLI
- label: I am building using file-based configuration (netlify.toml)
- label: I am building using file-based configuration (`netlify.toml`)
- type: dropdown
id: which-os
attributes:
Expand All @@ -51,8 +61,22 @@ body:
attributes:
label: Your `netlify.toml` file
description: (If you're using file-based config) Please provide a copy of your `netlify.toml` file.
render: shell
value: |
<details>
<summary>`netlify.toml`</summary>
```toml
# Paste content of your `netlify.toml` file here
```
</details>

- type: textarea
id: logs
attributes:
label: Relevant log output (or link to your logs)
value: |
<details>
<summary>Build logs</summary>
```
# Paste logs here
```
</details>
17 changes: 6 additions & 11 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,27 +1,22 @@
<!--Please tag yourself as the Assignee and at least one specific Ecosystem engineer as the Reviewer (@ascorbic, @lindsaylevine, @tiffanosaurus) -->
<!--Please tag yourself as the Assignee and netlify/integrations as the Reviewer -->

### Summary
<!-- Provide a summary brief summary of the change. -->

### Screenshots changes if applicable
<!-- Please delete this section if not applicable to this PR. -->

**Before:**

**After:**
<!-- Provide a brief summary of the change. -->

### Test plan

1. Visit the Deploy Preview ([insert link to specific page]())
...
1. Visit the Deploy Preview ([insert link to specific page]()) ...

### Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal (small picture)
### Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal

### Standard checks:

<!-- Please delete any options that reviewers shouldn't check. -->

- [ ] Check the Deploy Preview's Demo site for your PR's functionality
- [ ] Add docs when necessary

---

🧪 Once merged, make sure to update the version if needed and that it was published correctly.
4 changes: 0 additions & 4 deletions .prettierignore
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
# Markdown
*.md

# Logs
logs
*.log
Expand All @@ -19,6 +16,5 @@ node_modules
.parcel-cache

# Test
demo

Choose a reason for hiding this comment

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

[dust] I don't really see a reason why we should ignore demo since it should automatically lint anyway 🤷‍♀️

lib
tsconfig.json