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

--noGitHubIssueLinking option doesn't work properly #542

Assignees
Labels
bug Issue identified by VS Code Team member as probable bug
Milestone

Comments

@horihiro
Copy link

horihiro commented Mar 9, 2021

According to the following code, this.expandGitHubIssueLinks is set to true when the option is not only specified but also not specified.

expandGitHubIssueLinks: noGitHubIssueLinking,

vscode-vsce/src/package.ts

Lines 441 to 442 in f186af2

this.expandGitHubIssueLinks =
typeof options.expandGitHubIssueLinks === 'boolean' ? options.expandGitHubIssueLinks : true;

So GitHub issue links in Markdown files are always expanded regardless of the option.

@horihiro
Copy link
Author

horihiro commented Mar 9, 2021

The test for expandGitHubIssueLinks option of ReadmeProcessor exists, this test abehavior when expandGitHubIssueLinks is false as the following.

const processor = new ReadmeProcessor(manifest, { expandGitHubIssueLinks: false });

But as I commented, expandGitHubIssueLinks is always true as above.

@horihiro
Copy link
Author

Hi @joaomoreno , Could you please look into this? Thanks.

@joaomoreno joaomoreno self-assigned this Mar 12, 2021
@joaomoreno joaomoreno added the bug Issue identified by VS Code Team member as probable bug label Mar 12, 2021
@joaomoreno joaomoreno added this to the March 2021 milestone Mar 12, 2021
This was referenced Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment