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.

Sign up for GitHub

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

Jump to bottom

doc: remove Roadmap Working Group #9545

Closed
wants to merge 1 commit into from
Closed

doc: remove Roadmap Working Group #9545

wants to merge 1 commit into from

Conversation

williamkapke
Copy link
Contributor

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

See: nodejs/CTC#16

FYI: This will also be PRd to the Website.

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Nov 10, 2016
@williamkapke williamkapke mentioned this pull request Nov 10, 2016
Merged
bnoordhuis
bnoordhuis approved these changes Nov 10, 2016
View reviewed changes
Copy link
Member

@bnoordhuis bnoordhuis left a comment

Choose a reason for hiding this comment

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

LGTM if you s/Doc: Remove/doc: remove/ and s/See:/Refs:/ in the commit log.

@williamkapke williamkapke changed the title Doc: Remove Roadmap Working Group doc: remove Roadmap Working Group Nov 10, 2016
@williamkapke
Copy link
Contributor Author

@bnoordhuis's requested changes comeplete
(if I understood those replaces correctly heh)

Trott
Trott approved these changes Nov 11, 2016
View reviewed changes
Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

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

LGTM

mhdawson
mhdawson approved these changes Nov 11, 2016
View reviewed changes
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@silverwind
Copy link
Contributor

Patch does not land cleanly, can you rebase?

@williamkapke
Copy link
Contributor Author

@silverwind done

@silverwind
Copy link
Contributor

Thanks! Landed in fde607e.

@silverwind silverwind closed this Nov 12, 2016
silverwind pushed a commit that referenced this pull request Nov 12, 2016
@williamkapke @silverwind
doc: remove Roadmap Working Group
fde607e 
Refs: nodejs/CTC#16
PR-URL: #9545
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
addaleax pushed a commit that referenced this pull request Nov 22, 2016
@williamkapke @addaleax
doc: remove Roadmap Working Group
d09a9f4 
Refs: nodejs/CTC#16
PR-URL: #9545
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this pull request Dec 13, 2016
@williamkapke @MylesBorins
doc: remove Roadmap Working Group
06732ba 
Refs: nodejs/CTC#16
PR-URL: #9545
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this pull request Dec 13, 2016
@williamkapke @MylesBorins
doc: remove Roadmap Working Group
0088ed8 
Refs: nodejs/CTC#16
PR-URL: #9545
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this pull request Dec 21, 2016
@williamkapke @MylesBorins
doc: remove Roadmap Working Group
ed26229 
Refs: nodejs/CTC#16
PR-URL: #9545
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this pull request Dec 21, 2016
@williamkapke @MylesBorins
doc: remove Roadmap Working Group
4a49de5 
Refs: nodejs/CTC#16
PR-URL: #9545
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
This was referenced Dec 21, 2016
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mhdawson mhdawson mhdawson approved these changes

@cjihrig cjihrig cjihrig approved these changes

@lpinca lpinca lpinca approved these changes

@Trott Trott Trott approved these changes

@evanlucas evanlucas evanlucas approved these changes

@bnoordhuis bnoordhuis bnoordhuis approved these changes

@jbergstroem jbergstroem jbergstroem approved these changes

@silverwind silverwind silverwind approved these changes

Assignees
No one assigned
Labels
meta Issues and PRs related to the general management of the project.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.