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(tools): doc block links not clickable in VS Code #12336

Merged
merged 3 commits into from
Feb 9, 2021

Conversation

jogold
Copy link
Contributor

@jogold jogold commented Jan 4, 2021

Before

After


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@gitpod-io
Copy link

gitpod-io bot commented Jan 4, 2021

@@ -898,7 +898,7 @@ export default class CodeGenerator {
this.code.line('/**');
before.forEach(line => this.code.line(` * ${line}`.trimRight()));
if (link) {
this.code.line(` * @see ${link}`);
this.code.line(` * @link ${link}`);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I actually think that the right JSDoc should be {@link http://...}

@NGL321 NGL321 changed the title fix(cfn2ts): external links not clickable in VS Code fix(tools): cfn2ts external links not clickable in VS Code Jan 23, 2021
@NGL321 NGL321 added the package/tools Related to AWS CDK Tools or CLI label Jan 23, 2021
@rix0rrr rix0rrr changed the title fix(tools): cfn2ts external links not clickable in VS Code fix(tools): doc block links not clickable in VS Code Feb 9, 2021
rix0rrr
rix0rrr previously approved these changes Feb 9, 2021
@rix0rrr rix0rrr added the pr-linter/exempt-test The PR linter will not require test changes label Feb 9, 2021
@mergify mergify bot dismissed rix0rrr’s stale review February 9, 2021 11:04

Pull request has been modified.

@mergify
Copy link
Contributor

mergify bot commented Feb 9, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: aea0f00
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

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

@mergify
Copy link
Contributor

mergify bot commented Feb 9, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 4f17f92 into aws:master Feb 9, 2021
TLadd pushed a commit to TLadd/aws-cdk that referenced this pull request Feb 9, 2021
Before
<img src="https://user-images.githubusercontent.com/12623249/103578859-959d9600-4ed7-11eb-9879-7a6e86870bbd.png" width="500" />

After
<img src="https://user-images.githubusercontent.com/12623249/103578879-9f26fe00-4ed7-11eb-9a81-87a01c4a4115.png" width="500" />


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
NovakGu pushed a commit to NovakGu/aws-cdk that referenced this pull request Feb 18, 2021
Before
<img src="https://user-images.githubusercontent.com/12623249/103578859-959d9600-4ed7-11eb-9879-7a6e86870bbd.png" width="500" />

After
<img src="https://user-images.githubusercontent.com/12623249/103578879-9f26fe00-4ed7-11eb-9a81-87a01c4a4115.png" width="500" />


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package/tools Related to AWS CDK Tools or CLI pr-linter/exempt-test The PR linter will not require test changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants