Closed dmethvin closed 9 years ago
A few comments, but overall this looks good.
If there are no other comments I'll merge this.
@dmethvin @scottgonzalez No additional comment in a month. I guess we're happy with the proposed changes?
I'm happy with the changes.
Closed via 8071e5a3e8c8cd8ed7debd79c482e9baa2d52f81 and pushed via copy-foundation-docs
.
You did that backward. The change needs to happen in the foundation repo and then the update happens here via copy-foundation-docs
. I'm not sure that it really makes sense to keep this in the foundation repo though, so we might want to just maintain it here.
Ugh. I just assumed since it wasn't a private document that we'd do it here, and of course the PR was against the file here.
@dmethvin Can you delete the branch made for this PR? Also, what do you guys mean with the foundation repo
?
The foundation repo is a private one.
Per discussion at the EC meeting on Tuesday.