Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Add pre-commit config #1017

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

feat: Add pre-commit config #1017

wants to merge 4 commits into from
+1,953 −3,657

Commits on Aug 21, 2024

  1. feat: Add pre-commit config 

    * Add pre-commit/pre-commit-hooks for general linting.
   - Ignore CSV files.
* Add python-jsonschema/check-jsonschema for GitHub Actions.
  schema validation
* Add pre-commit.ci config.
   - Set autoupdate schedule to quarterly (the longest possible).
    @matthewfeickert
    matthewfeickert committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    521e4c3 View commit details
    Browse the repository at this point in the history

  2. style: Apply pre-commit hooks

    @matthewfeickert
    matthewfeickert committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    a9bb62f View commit details
    Browse the repository at this point in the history

  3. Add pre-commit/mirrors-prettier 

    Co-authored-by: Tetsuo Koyama <[email protected]>
    @matthewfeickert @tkoyama010
    matthewfeickert and tkoyama010 authored Aug 21, 2024
    Configuration menu
    Copy the full SHA
    9c28654 View commit details
    Browse the repository at this point in the history

  4. style: Apply prettier changes 

    * Also fix the following errors:

publisher/_static/scipy-proc.css
[error] publisher/_static/scipy-proc.css: SyntaxError: CssSyntaxError: Missed semicolon (28:23)
[error]   26 | .abstract {
[error]   27 |    width: 80ex;
[error] > 28 |    text-align: justify
[error]      |                       ^
[error]   29 |    padding-left: 1em;
[error]   30 | }
[error]   31 |
publisher/_static/screen.css
[error] publisher/_static/screen.css: SyntaxError: CssSyntaxError: Unknown word (57:3)
[error]   55 |   float: right;
[error]   56 |   margin: 0.5ex 10px 0.5ex 0px;
[error] > 57 |   padding 0;
[error]      |   ^
[error]   58 |   display: inline; /* forces IE to calc correct margin */
[error]   59 | }
[error]   60 |
    @matthewfeickert
    matthewfeickert committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    fe92e3c View commit details
    Browse the repository at this point in the history