Skip to content
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

Release 0.3.17 (template updates) - ?? #1076

Merged
merged 24 commits into from Feb 27, 2018
Merged
Show file tree
Hide file tree
Changes from 17 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
23 changes: 23 additions & 0 deletions .github/ISSUE_TEMPLATE.md
@@ -1,3 +1,19 @@
**Marked version:**

<!-- The NPM version or commit hash having the issue -->

<!--
If submitting something other than a defect with Marked itself, please use the following:
**Proposal type:** new feature | project operations | other
## What pain point are you perceiving?
## What solution are you suggesting?
-->

## Expectation

<!-- Describe the output you are expecting from marked -->
Expand All @@ -9,3 +25,10 @@
## What was attempted

<!-- Describe what code combination got you there -->

<!--
If error is thrown add the following:
## Call stack & console log
-->
55 changes: 38 additions & 17 deletions .github/PULL_REQUEST_TEMPLATE.md
@@ -1,25 +1,46 @@

<!--

If release PR, add ?template=release.md to the PR url to use the release PR template.

Otherwise, you are stating the this PR fixes an issue that has been submitted; or,
describes the issue or proposal under considersation.

-->

**Marked version:**

<!-- The NPM version or commit hash having the issue -->

## Description

<!-- describe what the PR does -->
- Fixes #### (if fixing a known issue; otherwise, describe issue using the following format)

<!--

## Expectation

Describe the output you are expecting from marked

## Result

Describe the output you received from marked

- Fixes ####
- Fixes list issues fixed by this PR
- Fixes will automatically close them once merged
## What was attempted

## Review
Describe what code combination got you there

### Submitter
-->

- [ ] All tests pass (CI should take care of this, once in place).
- [ ] All lint checks pass (CI should take care of this, once in place).
- Tests
- [ ] Test(s) exist to ensure functionality works (if no new tests added, list which tests cover this functionality).
- [ ] No tests required for this PR.
- [ ] Is release:
- [ ] Version in `package.json` has been updated (see [RELEASE.md](https://github.com/markedjs/marked/blob/master/RELEASE.md)).
- [ ] The `marked.min.js` has been updated; or,
- [ ] release does not change library.
## Submitter

### Reviewer
- [ ] Test(s) exist to ensure functionality and minimize regresstion (if no tests added, list tests covering this PR); or,
- [ ] no tests required for this PR.
- [ ] If submitting new feature, it has been documented in the appropriate places.

## Reviewer

??
- [ ] Draft GitHub release notes have been updated.
- [ ] case_insensitive_refs is only failing test (remove once CI is in place and all tests pass).
- [ ] All lint checks pass (remove once CI is in place).
- [ ] Merge PR
23 changes: 23 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE/release.md
@@ -0,0 +1,23 @@
## Submitter

- [ ] `$ npm version` has been run.
- [ ] Release notes in [draft GitHub release](https://github.com/markedjs/marked/releases) are up to date
- [ ] Reviewer checklist is complete.
- [ ] Merge PR.
- [ ] Publish GitHub release using `master` with correct version number.
- [ ] `$ npm publish` has been run.
- [ ] Create draft GitHub release to prepare next release.

Note: If merges to `master` occur after submitting this PR and before running `$ npm pubish` you should be able to

1. pull from `upstream/master` into the branch holding this version,
2. run `$ npm run build` to regenerate the `min` file, and
3. commit and push the updated changes.

## Reviewer

- [ ] Version in `package.json` has been updated (see [RELEASE.md](https://github.com/markedjs/marked/blob/master/RELEASE.md)).
- [ ] The `marked.min.js` has been updated; or,
- [ ] release does not change library.
- [ ] case_insensitive_refs is only failing test (remove once CI is in place and all tests pass).
- [ ] All lint checks pass (remove once CI is in place).