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 assorted tslint violations #131

Merged
merged 1 commit into from May 21, 2017

Conversation

Projects
None yet
2 participants
@c-w
Copy link
Contributor

commented May 17, 2017

No functional change, just letting VSCode fixup some files and extracting some methods to prevent variable shadowing.

This PR goes a long way towards resolving #13: we now only have 4 errors left, all of which are programming errors instead of style errors:

image

@morsh

This comment has been minimized.

Copy link
Contributor

commented May 18, 2017

@c-w - This is a blessed undertaking!!
I'll merge this first thing next week (Sunday) and make the appropriate test.

@morsh morsh merged commit 2ab21c7 into Azure:ibex-version-1.0 May 21, 2017

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