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: add TC meeting 2015-04-01 minutes #1371

Closed
wants to merge 0 commits into from
Closed

doc: add TC meeting 2015-04-01 minutes #1371

wants to merge 0 commits into from

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Apr 8, 2015

ref #1311

bnoordhuis
bnoordhuis reviewed Apr 8, 2015
View reviewed changes
doc/tc-meetings/2015-04-01.md
* Rod: requested that we not hold up progress in io.js pending movement on the JNAB / Foundation process and we should treat that as a separate thing that may or may not happen.
* Bert: happy with the changes but can’t +1 because of the TC company proportion counting
* Mikeal: just a miscount, needs to be changed, wanted to test if people are OK with this once we have more people on the TC. Main point was to flush out any concerns about raising the bar.
* Bert: suggested that it become 2 PRs, make the 2/3rds thing a separate issue.
Copy link
Member

Choose a reason for hiding this comment

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

s/Bert/Ben/

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed, thanks

@bnoordhuis
Copy link
Member

LGTM with comment.

Fishrock123
Fishrock123 reviewed Apr 8, 2015
View reviewed changes
doc/tc-meetings/2015-04-01.md
Ben: +1
Trevor: +1

### Proposal: Authorise @Fishrock123 to create releases [#1225](https://github.com/iojs/io.js/issues/1225)
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like there's 2 spaces between the header and issue link for some of these.

@Fishrock123
Copy link
Contributor

LGTM otherwise

@Fishrock123 Fishrock123 added the meta Issues and PRs related to the general management of the project. label Apr 8, 2015
rvagg added a commit that referenced this pull request Apr 9, 2015
@rvagg
doc: add TC meeting 2015-04-01 minutes
8ba9c4a 
Closes: #1311
PR-URL: #1371
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@rvagg rvagg closed this Apr 9, 2015
@rvagg
Copy link
Member Author

rvagg commented Apr 9, 2015

landed @ 8ba9c4a

@rvagg rvagg mentioned this pull request Apr 11, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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.
3 participants
@rvagg @bnoordhuis @Fishrock123