Skip to content

Upgrade tslint #1051

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

Merged
merged 5 commits into from
Apr 8, 2017
Merged

Upgrade tslint #1051

merged 5 commits into from
Apr 8, 2017

Conversation

afc163
Copy link
Member

@afc163 afc163 commented Mar 24, 2017

  • Upgrade tslint and tslint-react
  • Add tslint-eslint-config
  • Auto fix some errors can be fixed.
  • Manually fix left errors

Someboy come and finish the last task.

@mention-bot
Copy link

@afc163, thanks for your PR! By analyzing the history of the files in this pull request, we identified @silentcloud, @liqi07 and @warmhug to be potential reviewers.

@codecov
Copy link

codecov bot commented Mar 24, 2017

Codecov Report

Merging #1051 into master will decrease coverage by <.01%.
The diff coverage is 55.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1051      +/-   ##
==========================================
- Coverage   70.06%   70.06%   -0.01%     
==========================================
  Files         215      215              
  Lines        3979     3978       -1     
  Branches     1189     1188       -1     
==========================================
- Hits         2788     2787       -1     
  Misses       1189     1189              
  Partials        2        2
Flag Coverage Δ
#rn 73.14% <75%> (ø) ⬆️
#web 66.93% <40%> (-0.02%) ⬇️
Impacted Files Coverage Δ
components/card/CardHeader.tsx 100% <ø> (ø) ⬆️
components/modal/alert.web.tsx 30.76% <ø> (ø) ⬆️
components/pagination/index.tsx 80% <ø> (ø) ⬆️
components/badge/index.tsx 100% <ø> (ø) ⬆️
components/checkbox/Checkbox.tsx 82.35% <ø> (ø) ⬆️
components/steps/index.tsx 96.29% <ø> (ø) ⬆️
components/image-picker/index.tsx 60.86% <ø> (ø) ⬆️
components/checkbox/AgreeItem.tsx 85.71% <ø> (ø) ⬆️
components/drawer/index.tsx 85.71% <ø> (ø) ⬆️
components/pagination/index.web.tsx 79.48% <ø> (ø) ⬆️
... and 29 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7a23ab...5718d57. Read the comment docs.

@codecov
Copy link

codecov bot commented Mar 24, 2017

Codecov Report

Merging #1051 into master will increase coverage by 0.05%.
The diff coverage is 77.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1051      +/-   ##
==========================================
+ Coverage   69.79%   69.84%   +0.05%     
==========================================
  Files         218      218              
  Lines        4062     4079      +17     
  Branches     1212     1212              
==========================================
+ Hits         2835     2849      +14     
- Misses       1226     1229       +3     
  Partials        1        1
Flag Coverage Δ
#rn 72.03% <78.37%> (ø) ⬆️
#web 67.53% <76.47%> (+0.09%) ⬆️
Impacted Files Coverage Δ
components/search-bar/index.tsx 79.62% <ø> (ø) ⬆️
components/tab-bar/TabBarItem.tsx 50% <ø> (ø) ⬆️
components/card/CardFooter.web.tsx 100% <ø> (ø) ⬆️
components/card/CardFooter.tsx 100% <ø> (ø) ⬆️
components/drawer/index.tsx 85.71% <ø> (ø) ⬆️
components/modal/alert.web.tsx 30.76% <ø> (ø) ⬆️
components/switch/index.tsx 75% <ø> (ø) ⬆️
components/steps/index.tsx 96.29% <ø> (ø) ⬆️
components/card/CardHeader.tsx 100% <ø> (ø) ⬆️
components/modal/operation.web.tsx 23.8% <ø> (ø) ⬆️
... and 60 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd7faf6...d5cb14a. Read the comment docs.

@warmhug
Copy link
Contributor

warmhug commented Mar 28, 2017

这个改动尽快合并吧?不然冲突很多

@paranoidjk
Copy link
Contributor

这个我这周末处理掉。

@paranoidjk
Copy link
Contributor

wait for palantir/tslint-react#79

@paranoidjk
Copy link
Contributor

TypeScript is not 100% works fine with React.

DefinitelyTyped/DefinitelyTyped#11640

@paranoidjk paranoidjk changed the title [WIP] Upgrade tslint Upgrade tslint Apr 7, 2017
@paranoidjk paranoidjk force-pushed the upgrade-tslint branch 2 times, most recently from 219d83a to 4bb5618 Compare April 7, 2017 09:36
@paranoidjk
Copy link
Contributor

诶,本地是过的为啥ci挂了....

@paranoidjk paranoidjk merged commit ab73fc9 into master Apr 8, 2017
@paranoidjk paranoidjk deleted the upgrade-tslint branch April 8, 2017 05:14
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.

6 participants