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

Make Azure Pipelines use document scope #548

Merged
merged 1 commit into from
Nov 27, 2018
Merged

Make Azure Pipelines use document scope #548

merged 1 commit into from
Nov 27, 2018

Conversation

vtbassmatt
Copy link
Contributor

Hi. I'm a PM on Azure Pipelines. One of the features we've just shipped is an extension for VS Code to make it easier to author in our YAML-based format. For some reason, the same document parsed as YAML shows the right amount of GitLens entries, but when I switch over to treat it as Azure Pipelines, way too many entries get injected.

If there's a different way to solve this, I'm all for it! But this seemed to fix it properly in my local workspace settings, and I think it should be the default for GitLens users.

Thanks!

@eamodio eamodio changed the base branch from master to develop November 27, 2018 04:49
@eamodio eamodio merged commit 6a9d81b into gitkraken:develop Nov 27, 2018
@eamodio
Copy link
Member

eamodio commented Nov 27, 2018

@vtbassmatt Thanks for the contribution! This will make it into the GitLens 9 release coming very soon.

@eamodio eamodio added this to the Soon™ milestone Nov 27, 2018
@eamodio eamodio removed this from the Soon™ milestone Jul 19, 2019
ahmadalli added a commit to ahmadalli/vscode-gitlens that referenced this pull request Aug 6, 2019
it's the same as gitkraken#548 for ansible language
@ahmadalli ahmadalli mentioned this pull request Aug 6, 2019
8 tasks
eamodio pushed a commit that referenced this pull request Aug 12, 2019
it's the same as #548 for ansible language
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants