Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
58 changes: 29 additions & 29 deletions .codeclimate.yml
Original file line number Diff line number Diff line change
@@ -1,30 +1,30 @@
# Save as .codeclimate.yml (note leading .) in project root directory
version: "2"
languages:
Ruby: false
JavaScript: true
PHP: false
checks:
file-lines:
config:
threshold: 500
method-lines:
config:
threshold: 75
method-complexity:
config:
threshold: 10
similar-code:
config:
threshold: 65
plugins:
duplication:
enabled: true
config:
languages:
javascript:
mass_threshold: 110
count_threshold: 3
exclude_paths:
- "public/vendor/*"
# Save as .codeclimate.yml (note leading .) in project root directory
version: "2"
languages:
Ruby: false
JavaScript: true
PHP: false
checks:
file-lines:
config:
threshold: 500
method-lines:
config:
threshold: 75
method-complexity:
config:
threshold: 10
similar-code:
config:
threshold: 65
plugins:
duplication:
enabled: true
config:
languages:
javascript:
mass_threshold: 110
count_threshold: 3
exclude_paths:
- "public/vendor/*"
- "test/*"
24 changes: 12 additions & 12 deletions .devcontainer/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
FROM mcr.microsoft.com/devcontainers/javascript-node:1-22-bookworm

RUN apt-get update \
&& apt-get install -y \
iputils-ping \
redis-server \
&& rm -rf /var/lib/apt/lists/*

RUN curl https://qlty.sh | /bin/bash \
&& mv ~/.qlty/bin/qlty /usr/local/bin/qlty \
&& rm -rf ~/.qlty \
&& qlty --version
FROM mcr.microsoft.com/devcontainers/javascript-node:1-22-bookworm
RUN apt-get update \
&& apt-get install -y \
iputils-ping \
redis-server \
&& rm -rf /var/lib/apt/lists/*
RUN curl https://qlty.sh | /bin/bash \
&& mv ~/.qlty/bin/qlty /usr/local/bin/qlty \
&& rm -rf ~/.qlty \
&& qlty --version
20 changes: 10 additions & 10 deletions .dockerignore
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
.*
logs
test
node_modules
commitlint.config.js
nodebb.bat
renovate.json
*.yml
*.md
Dockerfile
.*
logs
test
node_modules
commitlint.config.js
nodebb.bat
renovate.json
*.yml
*.md
Dockerfile
16 changes: 8 additions & 8 deletions .editorconfig
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
root = true

[{*.js, *.css, *.tpl, *.json}]
indent_style = tab
end_of_line = lf
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = false
root = true
[{*.js, *.css, *.tpl, *.json}]
indent_style = tab
end_of_line = lf
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = false
24 changes: 12 additions & 12 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
# These files are text and should be normalized (convert crlf => lf)
*.json text
*.css text
*.less text
*.tpl text
*.html text
*.js text
*.md text

# Images should be treated as binary
# (binary is a macro for -text -diff)
*.png binary
# These files are text and should be normalized (convert crlf => lf)
*.json text
*.css text
*.less text
*.tpl text
*.html text
*.js text
*.md text
# Images should be treated as binary
# (binary is a macro for -text -diff)
*.png binary
*.jpg binary
94 changes: 47 additions & 47 deletions .github/ISSUE_TEMPLATE/p1-issue-form.yml
Original file line number Diff line number Diff line change
@@ -1,47 +1,47 @@
name: P1B Starter Task Issue
description: Use this template to describe the Qlty smell that you will refactor for P1B.
title: "P1B: Refactor (<full/path/to/file.js:line_number>): <smell>"
labels: ["P1B"]
body:
- type: markdown
attributes:
value: |
**Instructions:**
* Ensure no one else is working on the same **file + smell**.
* After submitting, comment **"I would like to work on this please!"** to trigger bot assignment.
* Keep evidence focused; do **not** paste a full project dump.

- type: checkboxes
id: uniqueness
attributes:
label: Uniqueness check
options:
- label: I have searched open issues and confirmed this **file + smell** is not already taken.
validations:
required: true

- type: input
id: filepath
attributes:
label: Full path to the JavaScript file
description: Must be under **src/** or **public/src/** and end with **.js**
placeholder: e.g., public/src/client/account/settings.js
validations:
required: true

- type: input
id: scope
attributes:
label: Function(s)/scope targeted
placeholder: e.g., render(), getCategoryTopics(), Flags.validate
validations:
required: true

- type: textarea
id: qlty_logs
attributes:
label: Relevant Qlty output
description: Run `qlty smells --no-snippet <file>` and paste only the lines that prove the smell and provide line numbers.
render: shell
validations:
required: true
name: P1B Starter Task Issue
description: Use this template to describe the Qlty smell that you will refactor for P1B.
title: "P1B: Refactor (<full/path/to/file.js:line_number>): <smell>"
labels: ["P1B"]
body:
- type: markdown
attributes:
value: |
**Instructions:**
* Ensure no one else is working on the same **file + smell**.
* After submitting, comment **"I would like to work on this please!"** to trigger bot assignment.
* Keep evidence focused; do **not** paste a full project dump.
- type: checkboxes
id: uniqueness
attributes:
label: Uniqueness check
options:
- label: I have searched open issues and confirmed this **file + smell** is not already taken.
validations:
required: true
- type: input
id: filepath
attributes:
label: Full path to the JavaScript file
description: Must be under **src/** or **public/src/** and end with **.js**
placeholder: e.g., public/src/client/account/settings.js
validations:
required: true
- type: input
id: scope
attributes:
label: Function(s)/scope targeted
placeholder: e.g., render(), getCategoryTopics(), Flags.validate
validations:
required: true
- type: textarea
id: qlty_logs
attributes:
label: Relevant Qlty output
description: Run `qlty smells --no-snippet <file>` and paste only the lines that prove the smell and provide line numbers.
render: shell
validations:
required: true
Loading
Loading