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

chore(deps-dev): bump typedoc to v0.23.23 #4458

Closed
wants to merge 32 commits into from

Conversation

MYoung25
Copy link
Contributor

Issue

#4134

Description

Upgrade TypeDoc to 0.23.x and implement identified a11y fixes in output

Testing

How was this change tested?

Additional context

Internal information in JS-3393, JS-4030, and CR-85945847


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

eduardomourar and others added 27 commits November 2, 2022 13:56
@MYoung25 MYoung25 added documentation This is a problem with documentation. dependencies This issue is a problem in a dependency. contribution/core This is a PR that came from AWS. labels Feb 22, 2023
@mtdowling
Copy link
Member

Is this really 32 commits, or is the branch out of sync with main?

@trivikr trivikr changed the title Chore/bump typedoc v0.23.23 chore(deps-dev): bump typedoc to v0.23.23 Feb 22, 2023
"typedoc": "0.19.2",
"typescript": "~4.6.2"
"typedoc": "^0.23.23",
"typescript": "4.7.3"
Copy link
Member

Choose a reason for hiding this comment

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

Can we bump typeScript in a different PR?

@@ -95,6 +95,10 @@
"react-native": {
"./dist-es/runtimeConfig": "./dist-es/runtimeConfig.native"
},
"typedoc": {
Copy link
Member

Choose a reason for hiding this comment

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

The entrypoint and readmeFile is defined in typedoc.json

Is this config in package.json redundnant?

Copy link
Contributor

Choose a reason for hiding this comment

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

this here is actually needed for the client SDK itself because they are generated in isolation and not in the context of the Yarn workspaces

@MYoung25
Copy link
Contributor Author

Closed in favor of #4515

@MYoung25 MYoung25 closed this Mar 13, 2023
srchase pushed a commit to srchase/smithy-typescript that referenced this pull request Mar 17, 2023
@github-actions
Copy link

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 Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
contribution/core This is a PR that came from AWS. dependencies This issue is a problem in a dependency. documentation This is a problem with documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants