Skip to content
This repository has been archived by the owner on Jan 19, 2019. It is now read-only.

[FIX] [indent] Add more indent cases + tests #274

Merged
merged 23 commits into from
Jan 9, 2019
Merged

[FIX] [indent] Add more indent cases + tests #274

merged 23 commits into from
Jan 9, 2019

Conversation

bradzacher
Copy link
Owner

@bradzacher bradzacher commented Jan 4, 2019 •
edited
Loading

  • Added many validation tests for our indent rule running on TS nodes.
  • Added an automatic generator to test the fixer as well as valid cases.
  • Handled more TS-specific node cases

@bradzacher bradzacher added the WIP PRs that are work in progress label Jan 4, 2019
@bradzacher bradzacher removed the WIP PRs that are work in progress label Jan 7, 2019
armano2
armano2 reviewed Jan 8, 2019
View reviewed changes
lib/rules/indent.js Outdated Show resolved Hide resolved
@bradzacher bradzacher mentioned this pull request Jan 8, 2019
Merged
@bradzacher bradzacher merged commit f612bdf into master Jan 9, 2019
@bradzacher bradzacher deleted the more-indent-cases branch January 9, 2019 01:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@armano2 armano2 armano2 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bradzacher @armano2