Skip to content

Conversation

@trivikr
Copy link
Member

@trivikr trivikr commented Oct 26, 2023

Issue

Fixes: #654

Description

Add check for StringLiteral to handle requires in TypeScript code

Testing

CI


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

@changeset-bot
Copy link

changeset-bot bot commented Oct 26, 2023

🦋 Changeset detected

Latest commit: c489d7d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
aws-sdk-js-codemod Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@trivikr trivikr changed the title Add STRING_LITERAL_TYPE_LIST Add check for StringLiteral to handle requires in TypeScript code Oct 26, 2023
@trivikr trivikr changed the title Add check for StringLiteral to handle requires in TypeScript code Check for StringLiteral type to handle requires in TypeScript code Oct 26, 2023
@trivikr trivikr marked this pull request as ready for review October 26, 2023 20:11
@trivikr trivikr requested a review from a team as a code owner October 26, 2023 20:11
@trivikr trivikr merged commit c72eac2 into main Oct 26, 2023
@trivikr trivikr deleted the literal-type-list branch October 26, 2023 20:19
@github-actions
Copy link
Contributor

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: The global require is not removed in TS file

1 participant