#3775 new defect

Introduce a code formatter into the tahoe

Reported by: fenn-cs Owned by:
Priority: normal Milestone: undecided
Component: unknown Version: n/a
Keywords: Cc:
Launchpad Bug:

Description

It would be nice to make code formatting a part of tahoe's linting process.

Our options are :

Change History (2)

comment:1 Changed at 2021-08-24T17:19:07Z by fenn-cs

Researched on code formatting approaches we can take for a large code base like Tahoe. Options include

  • Entire codebase reformat in giant commit (Mostly bad)
  • Module by module introduction
  • Progressive introduction (with a pre commit hook that enforces formatting rules on new files or change files)

https://blog.readme.com/adopting-coding-standards-within-a-large-codebase/

comment:2 Changed at 2021-08-24T21:32:58Z by meejah

A good branch to experiment with for how to bring an "old" branch up-to-date after a black-ening is: https://github.com/meejah/tahoe-lafs/tree/2916.grid-manager-proposal.5

(There are others with .4, etc that are older). The above probably still has "legitimate" conflicts that I'll need to address as master hasn't been merged into it in a while.

Note: See TracTickets for help on using tickets.