Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Adding SCMContentProvider implementation for TFVC #92

Merged
merged 1 commit into from
Feb 8, 2017

Conversation

jpricket
Copy link
Member

@jpricket jpricket commented Feb 8, 2017

User Story #884935

@msftclas
Copy link

msftclas commented Feb 8, 2017

Hi @jpricketMSFT, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Jason Prickett). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

const fsWatcher = workspace.createFileSystemWatcher("**");
const onWorkspaceChange = anyEvent(fsWatcher.onDidChange, fsWatcher.onDidCreate, fsWatcher.onDidDelete);
//const onTfvcChange = filterEvent(onWorkspaceChange, uri => /^\.tf\//.test(workspace.asRelativePath(uri)));
this.model = new Model(repoContext.RepoFolder, repoContext.TfvcRepository, onWorkspaceChange);
const onTfvcChange = filterEvent(onWorkspaceChange, uri => /^\$tf\//.test(workspace.asRelativePath(uri)));
Copy link
Contributor

Choose a reason for hiding this comment

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

Presumably we'll need to support .tf at some point...?

Copy link
Member Author

Choose a reason for hiding this comment

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

Agreed. Not sure what this even does yet.

@jpricket jpricket merged commit c2197b2 into master Feb 8, 2017
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.

3 participants