Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Nov 24 07:53

    JohanLarsson on nuget

    (compare)

  • Nov 24 07:53

    JohanLarsson on master

    Bump Gu.Roslyn.Extensions.Sourc… (compare)

  • Nov 24 07:53
    JohanLarsson closed #342
  • Nov 24 07:16
    dependabot[bot] labeled #342
  • Nov 24 07:16
    dependabot[bot] opened #342
  • Nov 24 07:16

    dependabot[bot] on nuget

    Bump Gu.Roslyn.Extensions.Sourc… (compare)

  • Nov 22 20:22

    JohanLarsson on master

    Fix tests. (compare)

  • Nov 22 10:28

    dependabot[bot] on nuget

    (compare)

  • Nov 22 10:28

    JohanLarsson on master

    Bump IDisposableAnalyzers from … (compare)

  • Nov 22 10:28
    JohanLarsson closed #341
  • Nov 22 10:27
    dependabot[bot] edited #341
  • Nov 22 10:26
    dependabot[bot] synchronize #341
  • Nov 22 10:26
    dependabot[bot] edited #341
  • Nov 22 10:26

    dependabot[bot] on nuget

    Bump IDisposableAnalyzers from … (compare)

  • Nov 22 10:26
    dependabot[bot] edited #341
  • Nov 22 10:26
    dependabot[bot] edited #341
  • Nov 22 10:25

    JohanLarsson on nuget

    (compare)

  • Nov 22 10:25

    JohanLarsson on master

    Bump Gu.Roslyn.Extensions.Sourc… (compare)

  • Nov 22 10:25
    JohanLarsson closed #340
  • Nov 22 10:25
    dependabot[bot] labeled #341
Johan Larsson
@JohanLarsson
we can always tweak if we find places where it is annoying
walking coerce is an option, not hard
brian-reichle
@brian-reichle
I'm thinking maybe 2 rules:
  • Rule 1: If the default value is null, then enforce that the property/getter is marked as nullable.
  • Rule 2: If the default value is non-null, enforce that the property/getter is not marked as nullable.
    This way people can select how strict they want to be.
Johan Larsson
@JohanLarsson
we are getting 1 in this release, maybe 2
do you mind creating an issue for 2?
No huge text needed, just for memory and discussion
I can see 2 being perhaps annoying
but it also makes sense given 1
and annoying is not a huge issue as it can be disabled
Johan Larsson
@JohanLarsson
image.png
pretty useful button :)
brian-reichle
@brian-reichle
I would like to see a 'clean .vs' :)
Johan Larsson
@JohanLarsson
what would it do?
brian-reichle
@brian-reichle
There have been many occasions where I have had to close VS, delete the .vs directory and restart VS, because VS sucks at invalidating invalid cache.
Johan Larsson
@JohanLarsson
yes, this is very true
close, git clean -fxd, open is kneejerk
brian-reichle
@brian-reichle
last night vs insisted that a particular member didn't exist, even while a full build passes without issue.
Johan Larsson
@JohanLarsson
vs has been a little better for me lately
brian-reichle
@brian-reichle
git clean -fxd isn't as convenient for me, some of the repositories I work in are too big and rebuilding the 100's of pre-req solutions gets expensive.
Johan Larsson
@JohanLarsson
crashes have not been so frequent
that has been somewhat frequent, not end of the world but annoying still
it claims it did not find it but still wipes the content
brian-reichle
@brian-reichle
I find that VS usually ignores me when I ask it to suppress in .editorconfig.
Johan Larsson
@JohanLarsson
quality softler :)
clean.vs is probably hard, vs locks stuff in there
Johan Larsson
@JohanLarsson
how big is the sln you are working in?
brian-reichle
@brian-reichle
most of the solutions are around 2-4 projects, though there are some outliers (like one solution with 18 projects). But then there are 100's of solutions in the repository (we use an in-house tool to build them all).
Johan Larsson
@JohanLarsson
bumping roslyn version all the way up to 3.7.0 for next release
Not sure what vs version it corresponds to
Johan Larsson
@JohanLarsson
severity error is a bit harsh for nullable
Johan Larsson
@JohanLarsson
Animation.gif
less annoying than I thought it would be
Johan Larsson
@JohanLarsson
works surprisingly well so far, updated a couple of libs
Johan Larsson
@JohanLarsson
regression I think
Johan Larsson
@JohanLarsson
makes sense to require nullable for both Ithink
Johan Larsson
@JohanLarsson
wrote them, can't wait forever, release now
Johan Larsson
@JohanLarsson
I'm changing all DiagnosticSeverity.Error to DiagnosticSeverity.Warning
Visual Studio does not handle error from third party analyzers well.
Bastian Schmidt
@batzen
Never had issues with error severity from third party analyzers
Johan Larsson
@JohanLarsson
Animation.gif
^ turned out nice & useful
Bastian Schmidt
@batzen
Hi there. Version 4 throws assembly load exceptions. Any hints on what else i might have to update?