Skip to content

issues Search Results · repo:G-Research/fsharp-analyzers language:F#

Filter by

20 results
 (70 ms)

20 results

inG-Research/fsharp-analyzers (press backspace or delete to remove)

G-Research/fsharp-analyzers
VirtualCall analyzer suggests quickfix on Collections.Generic.List

When using a function like Seq.head on a Collections.Generic.List, the suggested fix is to call the function on the List module, which does not accept the list from Collections.Generic, resulting in a ...
  • joprice
  • 1
  • Opened 
    on Jan 6
  • #86

G-Research/fsharp-analyzers
Support request: unexpected new analyzer failure when no relevant change was made

Hi - I ve got a new analyser failure on https://github.com/Smaug123/fsharp-prattparser/pull/9 which is a change completely unrelated to the analysers. I can t reproduce it locally on macOS. The failing ...
  • Smaug123
  • 5
  • Opened 
    on Mar 2, 2024
  • #82

G-Research/fsharp-analyzers
Potential false positive in GRA-LOGTEMPLMISSVALS-001

From the trenches: stepLog.LogWarning(sprintf Foo %dm %ds {Wasted}\n{Result} (int timespan.TotalMinutes) timespan.Seconds, timeWasteString, result) This gives Error GRA-LOGTEMPLMISSVALS-001 - The given ...
  • nojaf
  • Opened 
    on Feb 15, 2024
  • #79

G-Research/fsharp-analyzers
Colons should also be allowed inside log templates

From the trenches: logger.LogDebug( Blah {s_thing:l} foo {s_other_thing:l} , someString, someOtherString) See https://github.com/serilog/serilog/wiki/Formatting-Output Colons are allowed in this case. ...
  • nojaf
  • 1
  • Opened 
    on Feb 15, 2024
  • #78

G-Research/fsharp-analyzers
Expansion of dispose-before-async-is-run

I ve just raised https://github.com/G-Research/fsharp-analyzers/pull/74 which contains a failing test case reproducing a situation I encountered at work which the analysers currently do not catch: module ...
  • Smaug123
  • Opened 
    on Feb 14, 2024
  • #75

G-Research/fsharp-analyzers
Auto-publish to NuGet

Is there a reason we re currently leaving changes in unreleased state at all? (For example, https://github.com/G-Research/fsharp-analyzers/actions/runs/7726152791/job/21061944118 on commit 10808cb33f001365cc6c189a8099f164810cc3e9 ...
  • Smaug123
  • 2
  • Opened 
    on Feb 14, 2024
  • #73

G-Research/fsharp-analyzers
False positive on matching log args

I observed today in GR that GRA-LOGTEMPLMISSVALS-001 incorrectly fired on the following: log.LogWarning(e, Error: {Something} , 3) Since the first argument inherits from Exception, it should be ignored ...
  • Smaug123
  • 4
  • Opened 
    on Jan 30, 2024
  • #69

G-Research/fsharp-analyzers
Remove PartialAppAnalyzer

@Smaug123 Would you be okay with us removing the PartialAppAnalyzer? It s pretty much disabled everywhere and given recent events, the need to have it seems to be gone.
  • dawedawe
  • 1
  • Opened 
    on Jan 30, 2024
  • #68

G-Research/fsharp-analyzers
Inception

We should run the analyzers during CI on the analyzer code base.
  • nojaf
  • Opened 
    on Jan 29, 2024
  • #64

G-Research/fsharp-analyzers
Consider rearranging tests to fit in the IDE

If we moved data into the FSharp.Analyzers.Tests folder, and added the following to the test fsproj, then the test files would show up in the IDE: ItemGroup None Include= data\**\*.fs* / ...
  • Smaug123
  • 1
  • Opened 
    on Jan 24, 2024
  • #60
Issue origami icon

Learn how you can use GitHub Issues to plan and track your work.

Save views for sprints, backlogs, teams, or releases. Rank, sort, and filter issues to suit the occasion. The possibilities are endless.Learn more about GitHub Issues
ProTip! 
Restrict your search to the title by using the in:title qualifier.
Issue origami icon

Learn how you can use GitHub Issues to plan and track your work.

Save views for sprints, backlogs, teams, or releases. Rank, sort, and filter issues to suit the occasion. The possibilities are endless.Learn more about GitHub Issues
ProTip! 
Press the
/
key to activate the search input again and adjust your query.
Issue search results · GitHub