Skip to content

Conversation

@staabm
Copy link
Contributor

@staabm staabm commented Mar 12, 2018

as the README already is super long this would bring some help for first time users.

generated with http://tableofcontents.herokuapp.com/

@codecov
Copy link

codecov bot commented Mar 12, 2018

Codecov Report

Merging #618 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #618   +/-   ##
=========================================
  Coverage     81.52%   81.52%           
  Complexity      912      912           
=========================================
  Files            62       62           
  Lines          2078     2078           
=========================================
  Hits           1694     1694           
  Misses          384      384

@felixfbecker
Copy link
Owner

The ToC looks broken for me:

image

But besides that, frankly I think listing all the features in the ToC just to link to the screenshots bloats the README more than it simplifies it.

@staabm
Copy link
Contributor Author

staabm commented Mar 12, 2018

I simplified the TOC so the main points are mentioned.
do you like it like that?

@staabm
Copy link
Contributor Author

staabm commented May 13, 2018

ping @felixfbecker

@felixfbecker felixfbecker merged commit 26e3451 into felixfbecker:master May 14, 2018
@staabm staabm deleted the patch-1 branch May 14, 2018 05:40
RenovZ pushed a commit to RenovZ/php-language-server that referenced this pull request Mar 25, 2022
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.

2 participants