Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jan 19 10:45
    LoganDark commented #1340
  • Jan 11 11:18
    13k commented #984
  • Jan 10 16:06
    chrisduerr commented #984
  • Jan 10 15:27
    13k commented #984
  • Jan 01 01:10
    michaelkirk opened #1346
  • Dec 28 2020 17:01

    murarth on master

    Fix broken link in documentatio… (compare)

  • Dec 28 2020 17:01
    murarth closed #1345
  • Dec 28 2020 04:14
    atsheehan opened #1345
  • Dec 10 2020 23:45

    kchibisov on v0.26.0

    (compare)

  • Dec 10 2020 23:44

    kchibisov on master

    Release version 0.26.0 (compare)

  • Dec 10 2020 23:44
    kchibisov closed #1344
  • Dec 10 2020 19:23
    chrisduerr review_requested #1344
  • Dec 10 2020 19:23
    chrisduerr opened #1344
  • Dec 10 2020 17:58
    chrisduerr commented #1343
  • Dec 10 2020 17:38

    kchibisov on master

    Usa stable version of rustfmt f… (compare)

  • Dec 10 2020 17:38
    kchibisov closed #1343
  • Dec 10 2020 17:37
    kchibisov commented #1343
  • Dec 10 2020 17:19
    kchibisov review_requested #1343
  • Dec 10 2020 17:18
    kchibisov synchronize #1343
  • Dec 10 2020 17:17
    kchibisov opened #1343
matrixbot
@matrixbot
UndeadLeech on Freenode Generally I'd advice to ping Wayland testers and give them at least 24h. And if people regularly don't show up, remove them from the list of testers.
UndeadLeech on Freenode Not out of malice, just because it doesn't benefit anyone to have a list of testers where nobody is interested in testing.
kchibisov on Freenode Yeah, but the thing is that I don't have anything that I can impl in Wayland backend right now.
kchibisov on Freenode So the only things that can show up is purely source level bugs or something like that.
UndeadLeech on Freenode And bugs should be independently verified by two people.
kchibisov on Freenode Kai Mast: the fix for your bug got merged.
kchibisov on Freenode UndeadLeech: I guess,
UndeadLeech on Freenode Anything that changes the behavior of winit, should be tested by at least two people.
UndeadLeech on Freenode And if the capacity doesn't exist, it should be made transparent in the testers list.
UndeadLeech on Freenode I mean if two people tested the original code, it might not have existed in the first place.
kchibisov on Freenode That's true, but the only way I can change the winit behavior is adding new things, which I can't, because I've added everything I possible can for Wayland backend.
UndeadLeech on Freenode Fixing bugs changes the behavior of winit.
kchibisov on Freenode UndeadLeech: the original code was tested by more than just me.
UndeadLeech on Freenode The only changes that do not impact the behavior of winit are style changes.
UndeadLeech on Freenode kchibisov: It's not a guarantee of course.
kchibisov on Freenode Yeah, this is one is a bit wierd, since you should min/max size in window builder.
matrixbot
@matrixbot
UndeadLeech on Freenode I mean I'm not much of a winit maintainer, so I don't have much to say. But I would never allow any Alacritty PR that changes anything in the code without being tested by at least two people.
kchibisov on Freenode And this thing is pretty rare to happen, since there's
kchibisov on Freenode since there's no real value in setting those things unless you're writing some 'special' application.
kchibisov on Freenode Like you want to start your window in floating mode.
kchibisov on Freenode UndeadLeech: are you counting the original author or are you counting only testers?
UndeadLeech on Freenode At least one person besides the original author.
kchibisov on Freenode Ah, how convinient.
UndeadLeech on Freenode The same way we do it in Alacritty.
kchibisov on Freenode It just happend that there's always two ppl to test every patch in alacritty.
UndeadLeech on Freenode Yeah and according to the list of testers in winit there should be more than two people capable of testing Wayland patches.
matrixbot
@matrixbot
kchibisov on Freenode The thing with Wayland, that proper PRs testing is very hard, since you should check like 4 compositors.
kchibisov on Freenode And verify that something strange you're seeing is compositor bug.
kchibisov on Freenode Most of the time that's true for GNOME.
kchibisov on Freenode But yeah, for something more complex than this one liner, I'd ping folks and wait for day or two.
UndeadLeech on Freenode Perfect is the enemy of good. Merging PRs without any second party verifying them because "testing correctly is difficult" is not exactly constructive.
UndeadLeech on Freenode Once you have people verify things on at least one compositor, you can think about having multiple tested.
kchibisov on Freenode I mean if they verify on GNOME it's hard to say anything.
UndeadLeech on Freenode Why?
kchibisov on Freenode Like the amount of GNOME bugs I've seen is rediculuos.
UndeadLeech on Freenode We should be aware of how the application behaves. If something cannot be tested on GNOME then that should be stated ahead of time.
kchibisov on Freenode And it's not that hard to trigger them.
UndeadLeech on Freenode If there is a GNOME bug, we should be aware of it.
kchibisov on Freenode I mean, I report them to GNOME sometimes.
kchibisov on Freenode Like HiDPI on GNOME is completely broken last time I've tried.
UndeadLeech on Freenode So?
kchibisov on Freenode nothing, I guess.
Kai Mast Just tested the bugfix and it resolves the problem for me.
kchibisov on Freenode Kai Mast: yeah, just set min_inner_size if you really want it on a window and not in a builder.
UndeadLeech on Freenode If something applies only to certain compositors, that should be made clear from the start. If it should work on all compositors and it turns out during the testing process that it doesn't, that's exactly what testing is there for.
matrixbot
@matrixbot
kchibisov on Freenode I mean the thing with GNOME that it can arbitrary downscale you.
kchibisov on Freenode Like it just reverts scale factor after certain actions or something like that.
kchibisov on Freenode So I can't just state in every PR, 'please don't test anything on GNOME with HiDPI or verify logs in WAYLAND_DEBUG=1'.
kchibisov on Freenode I mean I agree that everything should be tested, I just don't want to test GNOME bugs.
matrixbot
@matrixbot
Kai Mast Would it help if I tested Gnome stuff for you?