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

Remove not required .tscparams #1358

Merged

Conversation

@vvakame
Copy link
Member

commented Nov 29, 2013

I hope they does not return to the --hasImplicitAny

Masahiro Wakame
@basarat

This comment has been minimized.

Copy link
Member

commented Nov 29, 2013

@borisyankov would appreciate @vvakame to have commit access so he can merge pull requests :)

basarat added a commit that referenced this pull request Nov 29, 2013
Remove not required .tscparams
@basarat basarat merged commit 4136501 into DefinitelyTyped:master Nov 29, 2013
1 check passed
1 check passed
default The Travis CI build passed
Details
@vvakame vvakame deleted the vvakame:remove-not-required-tscparams branch Nov 29, 2013
@vvakame

This comment has been minimized.

Copy link
Member Author

commented Nov 29, 2013

If you were to be given access, I also help to review & merge. 😺

@borisyankov

This comment has been minimized.

Copy link
Member

commented Nov 29, 2013

Sure, I just added you, Masahiro. Welcome ;)

On Fri, Nov 29, 2013 at 10:57 AM, Masahiro Wakame
notifications@github.comwrote:

If you were to be given access, I also help to review & merge. [image:
😺]


Reply to this email directly or view it on GitHubhttps://github.com//pull/1358#issuecomment-29503283
.

@vvakame

This comment has been minimized.

Copy link
Member Author

commented Nov 29, 2013

thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.