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

Migration to typescript #61

Merged
merged 23 commits into from
Aug 3, 2023
Merged

Migration to typescript #61

merged 23 commits into from
Aug 3, 2023

Conversation

thsaravana
Copy link
Member

Fixes #48

  • Migrated from JS to TS
  • Tested on both jacoco-playground and jacoco-android-playground
  • This is a basic JS to TS migration. We can do a lot of improvement to the code now. Those will be done later.

@thsaravana
Adding package.json value
b274bf0
@thsaravana
Simply converting all JS to TS
86859e5
@thsaravana
Main should be JS
dea714c
@thsaravana
Fix issue in index.ts
0acd661
@thsaravana
Print something in action()
01d00c7
@thsaravana
Trying a different import
4721a85
@thsaravana
More logs
afcfd25
@thsaravana
More logs
37825a5
@thsaravana
More logs
dae3756
@thsaravana
Fixing the util.test.ts
dd57529
@thsaravana
Migrating all tests to TS with the @ts-nocheck annotation
a29eeea
@thsaravana
Fix all issues in util.ts expect return types
3841f34
@thsaravana
Fix all issues in util.ts
5f8513c
@thsaravana
Fix all issues in render.ts
d971853
@thsaravana
Fix all issues in process.ts
be09c4c
@thsaravana
Fix some issues in action.ts
45882d7
@thsaravana
Fixing all issues in action.ts
de6c6e4
@thsaravana
Fixing all prettier issues
41e1865
@thsaravana
Fixing all errors
e1175a2
@thsaravana
Changed from 1.6.1
30e47aa
@thsaravana
Changed from 1.6.1 - Build
2e2856f
@thsaravana
Merge branch 'main' of github.com:Madrapps/jacoco-report into migrati…
1bd0382 
…on-to-typescript

# Conflicts:
#	__tests__/util.test.js
#	dist/index.js
#	src/action.ts
#	src/util.js
@thsaravana thsaravana added this to the 1.7 milestone Aug 3, 2023
@thsaravana thsaravana requested a review from instrap August 3, 2023 19:54
@thsaravana thsaravana merged commit 468b095 into main Aug 3, 2023
@thsaravana thsaravana deleted the migration-to-typescript branch August 3, 2023 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@instrap instrap instrap approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.7
Development

Successfully merging this pull request may close these issues.

Tech Debt - Migrate from JS to TS
2 participants
@thsaravana @instrap