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

Added support for baseUrl without paths #67

Merged
merged 1 commit into from
Nov 23, 2021
Merged

Added support for baseUrl without paths #67

merged 1 commit into from
Nov 23, 2021

Conversation

raouldeheer
Copy link
Contributor

Issues

This pull request should fix #63

Changes

  • Added support for baseUrl without paths.
  • Added test project for feature.

Conclusion

In the performance test is shown that the new feature doesn't add too much overhead.

Sources

Performance tests:

All results are an average of 5 test runs.
Test specs:

  • Ryzen 2700X CPU
  • 32GB Ram
  • 500GB Samsung 970 Evo SSD (With only the benchmark)
  • Node 17.1.0

With the new feature:

Result is time in ms.

AliasCount > 10 20 30 40 50 60 70 80 90 100
10 Files 13 11 11 10 10 12 13 12 14 16
20 Files 14 16 15 16 17 19 20 20 21 22
30 Files 18 19 22 22 25 26 27 29 31 32
40 Files 24 25 26 29 29 31 33 35 37 39
50 Files 28 30 34 40 37 38 45 45 46 49
60 Files 35 37 38 42 43 47 48 51 53 57
70 Files 40 42 45 49 51 54 56 59 63 66
80 Files 43 47 51 53 56 63 69 69 73 77
90 Files 49 52 54 58 63 68 74 79 83 86
100 Files 54 56 60 65 73 75 78 83 89 95

tsc-alias 1.4.1:

Result is time in ms.

AliasCount > 10 20 30 40 50 60 70 80 90 100
10 Files 13 10 10 11 10 12 13 13 14 16
20 Files 15 14 15 17 18 20 20 20 20 23
30 Files 19 20 21 25 27 27 29 32 30 33
40 Files 25 29 27 31 32 32 34 36 38 41
50 Files 30 31 34 35 42 42 43 44 46 50
60 Files 35 39 40 46 44 47 52 53 57 59
70 Files 38 42 43 48 50 53 57 61 64 67
80 Files 46 49 51 52 56 61 63 68 71 76
90 Files 49 53 54 57 62 69 72 74 78 83
100 Files 55 61 62 64 69 75 80 82 87 92

Added baseUrl resolving

Removed paths check

Added baseUrl only test project

Added .d.ts extensions

Optimized baseUrl resolving

Removed unnecessary variable
@justkey007 justkey007 merged commit 18b252a into justkey007:master Nov 23, 2021
@raouldeheer raouldeheer deleted the baseUrl-resolving branch November 30, 2021 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Support baseUrl without paths
2 participants