Skip to content

Update Readme.md#37

Open
isaacabraham wants to merge 1 commit into
fsprojects:masterfrom
isaacabraham:patch-1
Open

Update Readme.md#37
isaacabraham wants to merge 1 commit into
fsprojects:masterfrom
isaacabraham:patch-1

Conversation

@isaacabraham

Copy link
Copy Markdown

Explains that the extension is not coupled to a specific Fantomas version

Explains that the extension is not coupled to a specific Fantomas version
Comment thread Readme.md
- Fully asynchronous, does not increase project load times
- Loaded when required
- Respects key-bindings
- Not coupled to a specific version of Fantomas. Works with whichever version is installed on your machine

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggest to clarify that it works with whatever is installed in your repo? (which I think it does?)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I know the next line sort of clarifies but "installed on your machine" can be a bit misleading?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe mention it uses Fantomas.Client and link to this doc.

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