Skip to content

feat: add linter#433

Draft
alex-messer wants to merge 106 commits into
mainfrom
432-add-style-linter
Draft

feat: add linter#433
alex-messer wants to merge 106 commits into
mainfrom
432-add-style-linter

Conversation

@alex-messer

Copy link
Copy Markdown

#432

  • move all Style files from Less to Sass
  • add Pre-Commit Hoockst with Linter

ewoerner and others added 30 commits March 27, 2024 18:00
fix warning message and spanish translation #356 #361
Include link to new open source repo #336
The .gitattributes file was introduced in 179103b and set to force all
files into text mode with line-feed line endings. I believe this is the
wrong approach. It leads to files with Windows style line endings that
have existed in the repository priot to this being added being marked as
"wrong" by git and getting converted. For new clones of the repository,
they will show up as modified. The same happens for binary files such as
the favicons or a font.

This commit does two things:
- Add exceptions to the .gitattributes to exclude all types of binary
  files currently in the repository. This will not work if we
subsequently add oder ones, such as .jpg files.
- Change the line endings from crlf to lf in all files that were stil
  wrong.

More information on this can be found at
https://docs.github.com/en/get-started/getting-started-with-git/configuring-git-to-handle-line-endings.

Please note that this is not about personal preference, but to avoid new
contributors getting confused by "changed" files.
do not make git treat binary files as text
Hide account linking as it is currently not working
This commit introduces react-markdown and allows a small subset of basic
markdown syntax to be used in banner descriptions and warnings. It also
adds a new tooltip during banner creation that explains which parts of
markdown are allowed.

We explicitly only allow a small number of things. Images and
headings are not allowed, as they could break our page layout.
fix 'each child in list should have key' warning
Add Markdown for banner description and warning
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.1.3 to 5.1.7.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v5.1.7/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.1.7/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [braces](https://github.com/micromatch/braces) from 3.0.2 to 3.0.3.
- [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md)
- [Commits](micromatch/braces@3.0.2...3.0.3)

---
updated-dependencies:
- dependency-name: braces
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
…es-3.0.3

Bump braces from 3.0.2 to 3.0.3
KirDE and others added 13 commits October 12, 2024 21:26
Fix display of durations < 100 seconds
Bumps [eslint-plugin-import](https://github.com/import-js/eslint-plugin-import) from 2.29.0 to 2.31.0.
- [Release notes](https://github.com/import-js/eslint-plugin-import/releases)
- [Changelog](https://github.com/import-js/eslint-plugin-import/blob/main/CHANGELOG.md)
- [Commits](import-js/eslint-plugin-import@v2.29.0...v2.31.0)

---
updated-dependencies:
- dependency-name: eslint-plugin-import
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 20.11.19 to 22.7.7.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/node-22.7.7

Bump @types/node from 20.11.19 to 22.7.7
…nt-plugin-import-2.31.0

Bump eslint-plugin-import from 2.29.0 to 2.31.0
…or-small-banners

fix: center banner position on banner detail page
…e-redux

Refactor home page to not use Redux
Change account verification to the new COMM verification
@alex-messer alex-messer linked an issue Nov 8, 2024 that may be closed by this pull request
@alex-messer alex-messer changed the title feat: add linter WIP: feat: add linter Nov 8, 2024
@alex-messer alex-messer changed the title WIP: feat: add linter Draft feat: add linter Nov 8, 2024
@alex-messer alex-messer self-assigned this Nov 8, 2024
@alex-messer alex-messer marked this pull request as draft November 8, 2024 16:28
@alex-messer alex-messer changed the title Draft feat: add linter feat: add linter Nov 8, 2024
@alex-messer alex-messer marked this pull request as ready for review November 12, 2024 16:55
@alex-messer alex-messer marked this pull request as draft November 12, 2024 16:55
@ewoerner ewoerner force-pushed the main branch 5 times, most recently from 8faee5a to 2c306ad Compare February 17, 2026 22:11
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.

add Style Linter

4 participants