Skip to content

Conversation

@C123R
Copy link
Contributor

@C123R C123R commented Jun 5, 2022

Description of changes:

  • refactor readme
  • upgrade ts-node and typescript version to fix this issue
  • use default synthesizer to allow custom qualifier instead of hnb659fds.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

upgrade ts-node and typescript version

use default synthesizer for custom qualifer

Signed-off-by: Cizer Pereira <[email protected]>
@C123R C123R requested a review from kzarms June 5, 2022 16:40
kzarms
kzarms previously requested changes Jun 6, 2022
Copy link
Contributor

@kzarms kzarms left a comment

Choose a reason for hiding this comment

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

Let's discuss the purpose of the Default naming changes and fix linting errors.
The rest if fine. Thank you.

@kzarms kzarms dismissed their stale review June 10, 2022 12:09

Remove my blocker to unblock you. Verify with the team and make a desicion based on your judgment

@C123R C123R force-pushed the feat/refactor-readme branch from 7d83bdd to ca0c677 Compare June 13, 2022 08:23
@C123R C123R force-pushed the feat/refactor-readme branch from ca0c677 to 0efda29 Compare June 13, 2022 08:26
Copy link
Contributor

@kzarms kzarms left a comment

Choose a reason for hiding this comment

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

Looks great for me!
Thanks

@C123R C123R merged commit 447ea9d into main Jun 20, 2022
@kzarms kzarms deleted the feat/refactor-readme branch July 13, 2022 09:27
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.

update to npm v4.7.2, got error: False expression: Non-string value passed to ts.resolveTypeReferenceDirective

2 participants