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.

Sign up for GitHub

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

Jump to bottom

[WIP]Turn unused value warnings on by default #10546

Closed
wants to merge 15 commits into from
Closed

[WIP]Turn unused value warnings on by default #10546

wants to merge 15 commits into from

Commits on Mar 17, 2021

  1. Turn on unused warnings by default

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    33ec62c View commit details
    Browse the repository at this point in the history

  2. more nowarning in tests

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    b7e0494 View commit details
    Browse the repository at this point in the history

  3. Delete RedundantNewDiagnosticAnalyzer.fs

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    15e5840 View commit details
    Browse the repository at this point in the history

  4. update

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    4ca343f View commit details
    Browse the repository at this point in the history

  5. more updates

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    ddfe443 View commit details
    Browse the repository at this point in the history

  6. try this

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    219921a View commit details
    Browse the repository at this point in the history

  7. more

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    82fa658 View commit details
    Browse the repository at this point in the history

  8. again

    @cartermp
    cartermp committed Mar 17, 2021
    Copy the full SHA
    197b19a View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2021

  1. Merge branch 'main' into unused-warnings-on-default

    @vzarytovskii
    vzarytovskii committed Aug 2, 2021
    Copy the full SHA
    7de34c1 View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2021

  1. Remove 1182 from WarningsAsErrors, since we have it now enabled by de… 

    …fault, and treat warnings as errors by default
    @vzarytovskii
    vzarytovskii committed Aug 4, 2021
    Copy the full SHA
    11eeaa6 View commit details
    Browse the repository at this point in the history

  2. Add warnaserror to the FSharpBuild.Directory.Build.props, ignored 118… 

    …2 in one of the tests
    @vzarytovskii
    vzarytovskii committed Aug 4, 2021
    Copy the full SHA
    7b8ce6a View commit details
    Browse the repository at this point in the history

Commits on Aug 5, 2021

  1. Added --nowarn:1182 to the service tests, fixed an unused value in th… 

    …e unit test.
    @vzarytovskii
    vzarytovskii committed Aug 5, 2021
    Copy the full SHA
    ac05d5d View commit details
    Browse the repository at this point in the history

  2. Fix 1182 in F#QA tests, fix in service script tests

    @vzarytovskii
    vzarytovskii committed Aug 5, 2021
    Copy the full SHA
    a1a306b View commit details
    Browse the repository at this point in the history

  3. Add 1182 nowarn for cambridge suite

    @vzarytovskii
    vzarytovskii committed Aug 5, 2021
    Copy the full SHA
    f5f558b View commit details
    Browse the repository at this point in the history

Commits on Aug 6, 2021

  1. Added --nowarn:1182 by default, when running tests

    @vzarytovskii
    vzarytovskii committed Aug 6, 2021
    Copy the full SHA
    ee64b82 View commit details
    Browse the repository at this point in the history