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

Adds rfc template #316

Merged
merged 2 commits into from
Nov 3, 2016
Merged

Adds rfc template #316

merged 2 commits into from
Nov 3, 2016

Conversation

delambo
Copy link
Member

@delambo delambo commented Nov 2, 2016

We're testing a new rfc process in github Issues. So far the idea is to copy and use the rfc template from the Rust repository. We plan on working out the details for the process - how are rfcs labeled, what are their various stages, how are they accepted - as we roll out the first couple proposals.

Feel free to leave ideas here about the initial rfc process. I'll open an Issue so that we can further discuss the final process.

@tizmagik
Copy link
Contributor

tizmagik commented Nov 2, 2016

Consider issue templates

@delambo
Copy link
Member Author

delambo commented Nov 2, 2016

I'm not exactly sure how issue templates work, but from what I gather the text in the ISSUE_TEMPLATE is inserted into every new issue. If that's right, I'd prefer to leave it in a standalone file for now since I'd like to reserve rfcs for big stuff.

@tizmagik
Copy link
Contributor

tizmagik commented Nov 2, 2016

Right, but you don't have to insert the entire RFC template in the issue template, you can just point folks to it during issue creation behind HTML comment tags (so they don't show up in the actual issue). So, for example, something like this:

<!--
    Writing an RFC?: Please use the template in rfc/template.md

    Writing a bug report/feature request?: Please fill in the following fields:
-->    

### Versions of kyt, node, OS

### Bug or Feature?

### Current Behavior

### Desired Behavior

And so on...

@delambo
Copy link
Member Author

delambo commented Nov 2, 2016

Ah, got ya.

@delambo
Copy link
Member Author

delambo commented Nov 2, 2016

Got the issue template in: #317

Copy link
Contributor

@jaredmcdonald jaredmcdonald left a comment

Choose a reason for hiding this comment

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

LGTM

@ccpricenytimes ccpricenytimes merged commit 92f95a8 into master Nov 3, 2016
@ccpricenytimes ccpricenytimes deleted the adds-rfc-template branch November 3, 2016 18:56
tizmagik added a commit that referenced this pull request Dec 14, 2016
* master: (35 commits)
  creates local copies of packages so e2e tests use latest code (#345)
  adds style and script linter packages (#344)
  now using npm version to check for kyt-cli updates (#337)
  Adds kyt version option to setup command (#343)
  make lint-script command; lint now runs scripts and style (#339)
  Adds bootstrap scripts (#341)
  fixes test coverage command (#342)
  Run test from root directory (#336)
  Monorepo (#330)
  [ci skip] document Jest/Watchman issue and common workaround (#334)
  Catch SIGINT (#332)
  Fixes e2e tests (#326)
  Removes deprecated reflect eslint rule (#325)
  Upgrade Linter dependencies (#289)
  [doc]fix path of eslintrc and stylelintrc (#321)
  Update unit tests for changes made to resolve #303 (#318)
  adds ISSUE_TEMPLATE (#317)
  Adds rfc template (#316)
  document possible kyt setup repository url formats (#314)
  Overwrites default npm init test script on setup (#293)
  ...

# Conflicts:
#	.travis.yml
#	e2e_tests/tests/cli.test.js
#	package.json
#	packages/kyt-cli/cli/actions/setup.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants