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

update DCO to 1.1 #95

Merged
merged 1 commit into from Apr 29, 2016
Merged

update DCO to 1.1 #95

merged 1 commit into from Apr 29, 2016

Conversation

williamkapke
Copy link
Contributor

See #92.

@williamkapke
Copy link
Contributor Author

I'm not certain that the Express application should be changed- Can someone confirm this is OK..?

@jasnell
Copy link
Member

jasnell commented Apr 29, 2016

Yeah, I wouldn't change the Express application. That should be brought up to the express tc in an issue at http://github.com/expressjs/discussions

@williamkapke
Copy link
Contributor Author

Ok, Express application change removed. Can I get a LGTM?

@jasnell
Copy link
Member

jasnell commented Apr 29, 2016

LGTM

@williamkapke williamkapke merged commit 1a3adae into nodejs:master Apr 29, 2016
@williamkapke williamkapke mentioned this pull request Aug 4, 2016
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@williamkapke @jasnell