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

Add TypeScript Support #293

Closed
wants to merge 28 commits into from
Closed

Add TypeScript Support #293

wants to merge 28 commits into from

Conversation

vinitkumar
Copy link
Owner

@vinitkumar vinitkumar commented Jan 2, 2021 •
edited

Description

Issue Addressed (if applicable)

Issue number being fixed (Format: Fixes #xxx)

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Screenshots (if applicable):

This change is Reviewable

@lgtm-com
Copy link

lgtm-com bot commented Jan 2, 2021

This pull request introduces 2 alerts when merging 3e566f4 into c81e53f - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@vinitkumar vinitkumar changed the title Add TypeScrip Support Add TypeScript Support Jan 3, 2021
@lgtm-com
Copy link

lgtm-com bot commented Jan 3, 2021

This pull request introduces 1 alert when merging 7658f95 into c81e53f - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 3, 2021

This pull request introduces 2 alerts when merging f85392a into c81e53f - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 3, 2021

This pull request introduces 3 alerts when merging c878d81 into c81e53f - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 3, 2021

This pull request introduces 4 alerts when merging b442861 into c81e53f - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 4, 2021

This pull request introduces 3 alerts when merging d584f6a into c81e53f - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 4, 2021

This pull request introduces 4 alerts when merging c7b5532 into c81e53f - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 4, 2021

This pull request introduces 4 alerts when merging 4eb56cd into c81e53f - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@vinitkumar vinitkumar mentioned this pull request Jan 5, 2021
Open
@lgtm-com
Copy link

lgtm-com bot commented Jan 5, 2021

This pull request introduces 11 alerts when merging 3d19703 into c81e53f - view on LGTM.com

new alerts:

  • 11 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 10, 2021

This pull request introduces 11 alerts when merging 813720f into c81e53f - view on LGTM.com

new alerts:

  • 11 for Unused variable, import, function or class

@vinitkumar
fix: add a comment to delete a deprecated method
6b4d20e 
This is just for explanation

- Github Issue: #45

Reported-by: Edwin <edwin@solarplaza.com>
Authored-by: Vinit Kumar <mail@vinitkumar.me>
Signed-off-by: Vinit Kumar <mail@vinitkumar.me>
@vinitkumar
Merge branch 'develop' into feat/ts
99b870d
@lgtm-com
Copy link

lgtm-com bot commented Apr 27, 2021

This pull request introduces 11 alerts when merging 99b870d into c15df2e - view on LGTM.com

new alerts:

  • 11 for Unused variable, import, function or class

@vinitkumar vinitkumar closed this Dec 13, 2021
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.

None yet
1 participant
@vinitkumar