CareerCupid: While reviewing a merge request from a team mate who is working on a new feature, you find some piece of code that seems to work perfectly well, but some functions are written in a style that you don't particularly favor. Your team has a document for code style guidelines, but this case is not covered there. What do you do?
cupid.careers
CareerCupid helps you find the best people, jobs and companies to work with
  • Alex@lemmy.ml
    11·
    24 hours ago

    The link refers to it being a style in an area that isn’t covered. In that case I’d favour consistency with the current code it’s touching and open an issue to update the style guide to cover the new area.