Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Normalize the filepath so VS's error list has the correct interactions. #3145

Merged
merged 2 commits into from Aug 22, 2017
Merged

Normalize the filepath so VS's error list has the correct interactions. #3145

merged 2 commits into from Aug 22, 2017

Conversation

WGroenestein
Copy link
Contributor

Resolves #2977

@palantirtech
Copy link
Member

Thanks for your interest in palantir/tslint, @WGroenestein! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@WGroenestein
Copy link
Contributor Author

Only tested it in VS on windows

Copy link
Contributor

@adidahiya adidahiya left a comment

Choose a reason for hiding this comment

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

ignore the CI failure on container 1 (sorry about that, see #3127)

@@ -18,6 +18,7 @@
import { AbstractFormatter } from "../language/formatter/abstractFormatter";
import { IFormatterMetadata } from "../language/formatter/formatter";
import { RuleFailure } from "../language/rule/rule";
import * as path from "path";
Copy link
Contributor

Choose a reason for hiding this comment

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

the imports are not sorted so you have a lint error here.

@adidahiya adidahiya merged commit d63a434 into palantir:master Aug 22, 2017
HyphnKnight pushed a commit to HyphnKnight/tslint that referenced this pull request Apr 9, 2018
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.

None yet

3 participants