[PR #12] [MERGED] Lint #43

New issue

Closed

opened 2026-03-23 20:32:12 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 20:32:12 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/tubearchivist/browser-extension/pull/12
Author: @bakkot
Created: 12/3/2022
Status: ✅ Merged
Merged: 12/3/2022
Merged by: @bbilly1

Base: master ← Head: lint

📝 Commits (6)

2dc650c add linter and formatter
64d29b5 format
ee7da0e auto-fix lint
9c535d2 manually fix remaining lint errors
d01803f add github workflow to enforce linting of JS files
97deb21 add readme section about JS

📊 Changes

9 files changed (+2537 additions, -568 deletions)

View changed files

➕ .eslintrc.js (+21 -0)
➕ .github/workflows/lint_js.yml (+16 -0)
📝 .gitignore (+3 -0)
📝 README.md (+3 -0)
📝 extension/background.js (+144 -166)
📝 extension/popup.js (+129 -147)
📝 extension/script.js (+243 -255)
➕ package-lock.json (+1960 -0)
➕ package.json (+18 -0)

📄 Description

Basically the same deal as https://github.com/tubearchivist/tubearchivist/pull/345.

Includes a github action to enforce the linting in CI.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/tubearchivist/browser-extension/pull/12 **Author:** [@bakkot](https://github.com/bakkot) **Created:** 12/3/2022 **Status:** ✅ Merged **Merged:** 12/3/2022 **Merged by:** [@bbilly1](https://github.com/bbilly1) **Base:** `master` ← **Head:** `lint` --- ### 📝 Commits (6) - [`2dc650c`](https://github.com/tubearchivist/browser-extension/commit/2dc650c76b999491e89bec140bd8778bbb42c9e0) add linter and formatter - [`64d29b5`](https://github.com/tubearchivist/browser-extension/commit/64d29b5a4f1052f4352350671ce6b86f4f2779be) format - [`ee7da0e`](https://github.com/tubearchivist/browser-extension/commit/ee7da0e7267841b3a0f188ba524cad451f029d5e) auto-fix lint - [`9c535d2`](https://github.com/tubearchivist/browser-extension/commit/9c535d27e6397a1f0a5c457f3a7768bf18edb718) manually fix remaining lint errors - [`d01803f`](https://github.com/tubearchivist/browser-extension/commit/d01803f605532811d57eb1704fb26c5db1fb07cb) add github workflow to enforce linting of JS files - [`97deb21`](https://github.com/tubearchivist/browser-extension/commit/97deb2141c02d380c721fae6f0b99f6b6029a7ed) add readme section about JS ### 📊 Changes **9 files changed** (+2537 additions, -568 deletions) <details> <summary>View changed files</summary> ➕ `.eslintrc.js` (+21 -0) ➕ `.github/workflows/lint_js.yml` (+16 -0) 📝 `.gitignore` (+3 -0) 📝 `README.md` (+3 -0) 📝 `extension/background.js` (+144 -166) 📝 `extension/popup.js` (+129 -147) 📝 `extension/script.js` (+243 -255) ➕ `package-lock.json` (+1960 -0) ➕ `package.json` (+18 -0) </details> ### 📄 Description Basically the same deal as https://github.com/tubearchivist/tubearchivist/pull/345. Includes a github action to enforce the linting in CI. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>