Skip to content
Snippets Groups Projects
  1. Aug 11, 2023
    • Saurabh Kumar's avatar
      Add pre-commit with black,prettier and djhtml (#3502) · 4ce14204
      Saurabh Kumar authored
      - Add black, djhtml, prettier
      - Add precommit
      
      
      This PR add configurations for black, djhtml and prettier.
      
      - black is used for formatting python files
      - djhtml for formatting html files. Use `{# fmt: off #}` and `{# fmt: on
      #}` where needed.
      - prettier for formatting of sass,css,yml and js files
      
      Note: prettier picks up some of it's config from `.editorconfig`, so
      it's updated appropriately
      
      Formating of mardown files, minified or vendored js files are ignored.
      
      Formatting used off-the-self rules for better consistency with the
      community as large.
      
      The formatting is not applied to all the files as it will introduce a
      lot of merge conflict with
      current PRs, instead `pre-commit` config is provide which can be
      installed by `precommit install`
      locally. It will format only the modified files in the PR/commits. 
      
      Once majority of the current active PRs are either rebased & formatted
      using precommit or merged, we can
      run the `make fmt` on the full repo and activate additional prettier and
      black checks in our lint rules.
      
      The modified sass files fixes the eslint error that would came up if we
      were to apply the prettier formatter on them.
      
      Fixes #3500
      4ce14204
  2. Jul 22, 2022
  3. Feb 11, 2022
    • Daniel Schultz's avatar
      Add .editorconfig · 2b61141c
      Daniel Schultz authored
      Editorconfig is a standard that helps editors know what whitespace rules
      to follow for a given project.
      
      For more information, see:
      
      https://editorconfig.org/
      
      These rules were determined based on feedback from project maintainers.
      
      There are some files that use different numbers of spaces, but these
      files were imposed on the project externally.  We use 4 everywhere we
      touch.
      
      Issue #2716
      2b61141c
Loading