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

Return type should go on next line #3041

Closed
4 tasks
nojaf opened this issue Jan 31, 2024 · 0 comments · Fixed by #3045
Closed
4 tasks

Return type should go on next line #3041

nojaf opened this issue Jan 31, 2024 · 0 comments · Fixed by #3045

Comments

@nojaf
Copy link
Contributor

nojaf commented Jan 31, 2024

Issue created from fantomas-online

Code

type FSharpChecker with

    member this.ParseAndCheckDocument
        (
            filePath: string,
            sourceText: string,
            options: FSharpProjectOptions,
            allowStaleResults: bool
        ) : Async<(FSharpParseFileResults * ParsedInput * FSharpCheckFileResults) option> =
        ()

Result

type FSharpChecker with

    member this.ParseAndCheckDocument
        (filePath: string, sourceText: string, options: FSharpProjectOptions, allowStaleResults: bool) : Async<
                                                                                                             (FSharpParseFileResults *
                                                                                                             ParsedInput *
                                                                                                             FSharpCheckFileResults) option
                                                                                                          >
        =
        ()

Problem description

Related to the changes of #3038.
Max line length is not respected here. The return type should go on the next line.

Extra information

  • The formatted result breaks my code.
  • The formatted result gives compiler warnings.
  • I or my company would be willing to help fix this.
  • I would like a release if this problem is solved.

Options

Fantomas main branch at 1/1/1990

Default Fantomas configuration

Did you know that you can ignore files when formatting by using a .fantomasignore file?
PS: It's unlikely that someone else will solve your specific issue, as it's something that you have a personal stake in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant