• shastaxc@lemm.ee
    link
    fedilink
    arrow-up
    7
    ·
    22 hours ago

    Having dealt with this drama before, and as team lead currently, as soon as this project began, the first thing I did was set up a linter with very specific rules. If a dev doesn’t agree with the rules defined there, too bad. They’re there to prevent this kind of drama, and to make peer reviews more focused and productive. I don’t wanna see PRs about syntax. They’re nearly always just a difference of opinion.

    • CanadaPlus@lemmy.sdf.org
      link
      fedilink
      arrow-up
      4
      ·
      22 hours ago

      If I ever get around to writing my own language, it’s going to take parse trees off the disk, so people can deparse them into whatever they want and STFU.

  • dumbass@leminal.space
    link
    fedilink
    English
    arrow-up
    2
    arrow-down
    1
    ·
    edit-2
    21 hours ago

    When I was a mod on Reddit we had a slack channel, one of the new mods got outed and shamed for a post where they were a dick at a show, the sub turned on them so we had to remove them, the head mod scolded them while we were all silent watching, it felt like we were kids hiding on the top of the stairs watching a sibling get scolded.

  • MajorHavoc@programming.dev
    link
    fedilink
    arrow-up
    7
    ·
    edit-2
    9 hours ago

    Oof. This is a pet peeve of mine.

    As manager, I shut this conversation down via direct messages to each team member involved.

    I remind them that they agreed during retro to live with the current set of decisions for exactly two weeks until next retro.

    I don’t dictate much, but I do dictate that Slack isn’t an acceptable place for this kind of discussion, on my team.

    The only related thing, that belongs in slack, on my team, is a link to the current accepted team standard - which will be open for review and changes again during next retro.

    Alternately, if there’s no standard for this yet then my team knows they’re encouraged to wing it until we discuss at next retro.

    And yeah, I’ve had to open an issue to revisit a variable name after retro, lol.

    My team are an opinionated bunch, and they’re often perfectionists.

    • AwkwardLookMonkeyPuppet@lemmy.world
      link
      fedilink
      English
      arrow-up
      1
      ·
      10 hours ago

      You allow naming schemes to change every two weeks? That’s just insane! You might as well not have a naming convention then, since the project is going to be full of different conventions.

      • MajorHavoc@programming.dev
        link
        fedilink
        arrow-up
        2
        ·
        edit-2
        9 hours ago

        You might as well not have a naming convention then, since the project is going to be full of different conventions.

        Oh, I skipped this. Lol. Obviously not. As a team, they can implement whatever convention change they want, every two weeks.

        As manager, I expect them to update all active projects, in their entirety, to the new convention, each time.

        And as I mentioned in my other comment, if their test coverage isn’t at a level that makes me confident in that kind of global change (70% tends to be plenty), then I reserve the right to table it - until they bring the test coverage up (on all impacted projects).

  • bleistift2@sopuli.xyz
    link
    fedilink
    English
    arrow-up
    12
    ·
    2 days ago

    Let me tell you: I’m currently developing a user-defined, recursive form, and most of the trouble with that stems from the fact that I don’t have a good name for the repeatable part of that form (as opposed to a static part) and the thing the form is embedded in.

    Variable names do matter.

  • pfm@scribe.disroot.org
    link
    fedilink
    arrow-up
    4
    ·
    2 days ago

    I don’t get this meme at all… What am I expected to see in this picture? Or how am I supposed to interpret it?