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

Fix inactivityTimeout declaration. Fixes #2298. #2299

Closed

Conversation

pavelhoral
Copy link
Contributor

For some reason contrib feature submit (as described in CONTRIBUTING.md) did not work so I am submitting this manually via GitHub web.

@mmcc
Copy link
Member

mmcc commented Jul 1, 2015

👍 Looks good to me.

@heff
Copy link
Member

heff commented Jul 2, 2015

lgtm

Thanks for trying out contrib. We need to fix the contributing guide and that process.

@heff heff closed this in 26a7862 Jul 2, 2015
@heff
Copy link
Member

heff commented Jul 2, 2015

Merged in, thanks! What error did you see with contrib feature submit?

@pavelhoral
Copy link
Contributor Author

I did few steps wrong at first (e.g. I didn't checkout the feature branch). However in the end everything works... just the pull request was not created. I get no error from contrib feature submit. After asking pull request details and GH credentials it looked like it is doing something, but ended up making nothing (the only output was something like -> undefined, which was probably indication that something failed).

@pavelhoral
Copy link
Contributor Author

@heff It is already reported as contrib/contrib#22 .

@heff
Copy link
Member

heff commented Jul 3, 2015

Ah, got it. Thanks!

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.

3 participants