Skip to content

Conversation

@filipchristiansen
Copy link
Contributor

@filipchristiansen filipchristiansen commented Jul 4, 2025

  • Add eslint to pre-commit hooks
  • getFileName + toggleFile moved from result.jinja to utils.js
  • Run linter

…sult.jinja`

* Add `eslint` to `pre-commit` hooks
* Remove `result.jinja`
    * `getFileName` + `toggleFile` moved to `utils.js`
* `getFileName` + `toggleFile` moved to `utils.js`
* leftover `<div data-results>` extracted to `_macros.jinja` as `results_container()`
* Run linter
@filipchristiansen filipchristiansen changed the title chore(pre-commit) / refactor(templates): add ESLint pre-commit hook + remove result.jinja chore(pre-commit) / refactor(templates): add ESLint pre-commit hook + refactor result.jinja Jul 4, 2025
@ix-56h
Copy link
Contributor

ix-56h commented Jul 4, 2025

lgtm

@filipchristiansen filipchristiansen merged commit dead917 into main Jul 4, 2025
18 checks passed
@filipchristiansen filipchristiansen deleted the eslint branch July 4, 2025 16:32
BareninVitalya pushed a commit to BareninVitalya/gitingest that referenced this pull request Jul 5, 2025
… + refactor `result.jinja` (coderamp-labs#379)

* chore(pre-commit) / refactor(templates): add ESLint hook + refactor `result.jinja`

* Add `eslint` to `pre-commit` hooks
* `getFileName` + `toggleFile` moved to `utils.js`
* Run linter
ix-56h pushed a commit to ix-56h/gitingest that referenced this pull request Jul 7, 2025
… + refactor `result.jinja` (coderamp-labs#379)

* chore(pre-commit) / refactor(templates): add ESLint hook + refactor `result.jinja`

* Add `eslint` to `pre-commit` hooks
* `getFileName` + `toggleFile` moved to `utils.js`
* Run linter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants