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

Dashboard: format frontend code #1235

Merged
merged 31 commits into from
Jul 17, 2020

Conversation

Gerhut
Copy link
Member

@Gerhut Gerhut commented Jul 8, 2020

Big changes, will merge after 1.7 release.



Technically all fixes are equivalent, filter out the .js, .ts, .tsx to see if there are any incorrect configurations.

@Gerhut Gerhut force-pushed the dashboard/frontend/lint branch from 398e135 to ef98932 Compare July 9, 2020 06:56
@Gerhut Gerhut marked this pull request as ready for review July 9, 2020 06:56
@Gerhut Gerhut marked this pull request as draft July 9, 2020 09:30
@Gerhut Gerhut force-pushed the dashboard/frontend/lint branch from 800982b to 546ac19 Compare July 12, 2020 13:52
@coveralls
Copy link

coveralls commented Jul 12, 2020

Pull Request Test Coverage Report for Build 3701

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.221%

Totals Coverage Status
Change from base Build 3696: 0.0%
Covered Lines: 968
Relevant Lines: 1038

💛 - Coveralls

@Gerhut Gerhut marked this pull request as ready for review July 12, 2020 13:56
@Gerhut Gerhut requested a review from Anbang-Hu July 12, 2020 14:07
@Gerhut Gerhut force-pushed the dashboard/frontend/lint branch from b3073eb to 3f08c30 Compare July 16, 2020 01:27
@Gerhut Gerhut merged commit 61b9d7f into microsoft:dltsdev Jul 17, 2020
@Gerhut Gerhut deleted the dashboard/frontend/lint branch July 17, 2020 03:13
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.

3 participants