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

fix(client-sts): duplicate stripInternal entry in tsconfig #2545

Merged
merged 1 commit into from
Jul 21, 2021

Conversation

aaroncowie
Copy link
Contributor

@aaroncowie aaroncowie commented Jul 6, 2021

Issue

Issue number, if available, prefixed with "#"

No issue

Description

What does this implement/fix? Explain your changes.

This caused a warning for me when bundling with esbuild

> node_modules/@aws-sdk/client-ecs/node_modules/@aws-sdk/client-sts/tsconfig.json:26:4: warning: Duplicate key "stripInternal" in object literal
    26 │     "stripInternal": true,
       ╵     ~~~~~~~~~~~~~~~

Testing

How was this change tested?

Not tested, change made through github web ui

Additional context

Add any other context about the PR here.


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

This caused an error for me when bundling with esbuild
> node_modules/@aws-sdk/client-ecs/node_modules/@aws-sdk/client-sts/tsconfig.json:26:4: warning: Duplicate key "stripInternal" in object literal
    26 │     "stripInternal": true,
       ╵     ~~~~~~~~~~~~~~~
@aaroncowie aaroncowie changed the title Fixed duplicate "stripInternal" entry in client-sts tsconfig fix(client-sts): Fixed duplicate "stripInternal" entry in client-sts tsconfig Jul 6, 2021
@aaroncowie
Copy link
Contributor Author

Feel free to reject this pull request and push the change separately, just wanted to flag this as an issue.

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@a51c2f4). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2545   +/-   ##
=======================================
  Coverage        ?   60.24%           
=======================================
  Files           ?      513           
  Lines           ?    27359           
  Branches        ?     6577           
=======================================
  Hits            ?    16482           
  Misses          ?    10877           
  Partials        ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a51c2f4...913b677. Read the comment docs.

@aws-sdk-js-automation
Copy link

AWS CodeBuild CI Report

  • CodeBuild project: sdk-staging-test
  • Commit ID: 913b677
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@AllanZhengYP AllanZhengYP changed the title fix(client-sts): Fixed duplicate "stripInternal" entry in client-sts tsconfig fix(client-sts): duplicate "stripInternal" entry in client-sts tsconfig Jul 16, 2021
@AllanZhengYP AllanZhengYP changed the title fix(client-sts): duplicate "stripInternal" entry in client-sts tsconfig fix(client-sts): duplicate stripInternal entry in tsconfig Jul 21, 2021
@AllanZhengYP AllanZhengYP merged commit e7bcbbc into aws:main Jul 21, 2021
@github-actions
Copy link

github-actions bot commented Aug 5, 2021

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 Aug 5, 2021
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.

None yet

4 participants