Skip to content

Conversation

@ParthS007
Copy link

No description provided.

@ParthS007 ParthS007 force-pushed the plugin-loader-typescript branch from 8ce542b to 9a55f89 Compare June 15, 2020 21:42
@ParthS007 ParthS007 marked this pull request as ready for review June 15, 2020 21:44
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (1.8.0_242) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

Copy link
Member

@1000TurquoisePogs 1000TurquoisePogs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hope that this exercise helped you to understand typescript more. the PR contains bugs, so please look at my comments to learn more

}
}

const plugInConstructorsByType = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is very similar in concept to an enum, so maybe it should be made into one.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please check now, there were some errors in below makePlugin after adding enum.

Signed-off-by: Parth Shandilya <[email protected]>
@ParthS007 ParthS007 force-pushed the plugin-loader-typescript branch from bee2bac to c209e93 Compare June 22, 2020 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants