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

Add Typescript language module #1260

Merged
merged 26 commits into from
Sep 6, 2023
Merged

Add Typescript language module #1260

merged 26 commits into from
Sep 6, 2023

Conversation

Kr0nox
Copy link
Member

@Kr0nox Kr0nox commented Aug 23, 2023

This PR adds a language module for parsing TypeScript and JavaScript.
Since the grammar specifies no version, but code from TS v5 can be parsed (tho no tokens are extracted) the version is estimated to be v5.

The token extraction is inspired by Java and Python

@Kr0nox Kr0nox changed the title Add Files for typescript Add Typescript language module Aug 23, 2023
@tsaglam tsaglam added enhancement Issue/PR that involves features, improvements and other changes major Major issue/feature/contribution/change language PR / Issue deals (partly) with new and/or existing languages for JPlag labels Aug 24, 2023
@tsaglam tsaglam requested review from a team August 24, 2023 08:01
README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@TwoOfTwelve TwoOfTwelve left a comment

Choose a reason for hiding this comment

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

Looks pretty good.

@dfuchss
Copy link
Member

dfuchss commented Sep 5, 2023

If the last issue is resolved, I think the PR would be ready to merge.

Is this correct @Kr0nox ? :)

@Kr0nox
Copy link
Member Author

Kr0nox commented Sep 5, 2023

@sebinside wanted to have a look at the choice of tokens. I dont know if he did that already

@sebinside sebinside self-requested a review September 5, 2023 14:58
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 6, 2023

[JPlag Plagiarism Detector] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

79.4% 79.4% Coverage
3.1% 3.1% Duplication

@dfuchss dfuchss merged commit 48acddb into develop Sep 6, 2023
@dfuchss dfuchss deleted the feature/ts-lang branch September 6, 2023 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue/PR that involves features, improvements and other changes language PR / Issue deals (partly) with new and/or existing languages for JPlag major Major issue/feature/contribution/change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants