8000 chore: setup knip and remove unused exports, files, and dependencies by brettkolodny · Pull Request #17608 · coder/coder · GitHub
[go: up one dir, main page]

Skip to content

chore: setup knip and remove unused exports, files, and dependencies #17608

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.

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

Merged
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
a75993a
chore: add knip as dev dep and create config
brettkolodny Apr 29, 2025
d947dc7
chore: update knip config to ignore exported types and interfaces
brettkolodny Apr 29, 2025
a23554c
chore: add unused imports to biome config
brettkolodny Apr 29, 2025
cf48a1b
chore: remove unused imports
brettkolodny Apr 29, 2025
1e2abbf
chore: remove unused exports
brettkolodny Apr 29, 2025
acb2dd3
chore: remove unused exports
brettkolodny Apr 29, 2025
952015b
chore: remove unused exports
brettkolodny Apr 29, 2025
fd1b1a6
chore: remove unused exports
brettkolodny Apr 29, 2025
f45a9c2
chore: remove duplicate exports
brettkolodny Apr 29, 2025
7b66bb4
chore: remove duplicate exports
brettkolodny Apr 29, 2025
4388699
chore: remove duplicate exports
brettkolodny Apr 29, 2025
85e845b
chore: remove unused dependencies
brettkolodny Apr 29, 2025
195a7e6
chore: remove unused script
brettkolodny Apr 29, 2025
28b3394
chore: update knip config to ignore protoc binary 8000
brettkolodny Apr 29, 2025
c0ceb87
chore: remove unused files
brettkolodny Apr 29, 2025
e8b672e
chore: remove unused dev dependencies
brettkolodny Apr 29, 2025
662d812
chore: remove unused dev dependencies
brettkolodny Apr 29, 2025
e50ca41
chore: remove unused dev dependencies
brettkolodny Apr 29, 2025
2619b00
chore: remove unused dev dependencies
brettkolodny Apr 29, 2025
25ccd6c
Merge branch 'main' into brett-i600/add-knip-to-coder-coder-and-remov…
brettkolodny Apr 29, 2025
3c097a4
fix: re-add ts-proto as a dev dep
brettkolodny Apr 29, 2025
db6c7b5
Merge branch 'main' into brett-i600/add-knip-to-coder-coder-and-remov…
brettkolodny Apr 30, 2025
501187e
chore: move serviceWorker.ts to knip entry
brettkolodny Apr 30, 2025
b51d7f8
chore: add e2e files as knip project
brettkolodny Apr 30, 2025
c509697
chore: remove unused exports
brettkolodny Apr 30, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
chore: remove unused exports
  • Loading branch information
brettkolodny committed Apr 30, 2025
commit c50969775005cbf5a4d44a88ff7740ac5e41b95a
8 changes: 0 additions & 8 deletions site/e2e/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -78,14 +78,6 @@ export const premiumTestsRequired = Boolean(

export const license = process.env.CODER_E2E_LICENSE ?? "";

/**
* Certain parts of the UI change when organizations are enabled. Organizations
* are enabled by a license entitlement, and license configuration is guaranteed
* to run before any other tests, so having this as a bit of "global state" is
* fine.
*/
export const organizationsEnabled = Boolean(license);

// Disabling terraform tests is optional for environments without Docker + Terraform.
// By default, we opt into these tests.
export const requireTerraformTests = !process.env.CODER_E2E_DISABLE_TERRAFORM;
Expand Down
4 changes: 2 additions & 2 deletions site/e2e/helpers.ts
8000
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ export async function login(page: Page, options: LoginOptions = users.owner) {
(ctx as any)[Symbol.for("currentUser")] = options;
}

export function currentUser(page: Page): LoginOptions {
function currentUser(page: Page): LoginOptions {
const ctx = page.context();
// biome-ignore lint/suspicious/noExplicitAny: get the current user
const user = (ctx as any)[Symbol.for("currentUser")];
Expand Down Expand Up @@ -875,7 +875,7 @@ export const echoResponsesWithExternalAuth = (
};
};

export const fillParameters = async (
const fillParameters = async (
page: Page,
richParameters: RichParameter[] = [],
buildParameters: WorkspaceBuildParameter[] = [],
Expand Down
8 changes: 2 additions & 6 deletions site/package.json
Original file line number Diff line number Diff line change
75AF Expand Up @@ -192,11 +192,7 @@
"vite-plugin-checker": "0.8.0",
"vite-plugin-turbosnap": "1.0.3"
},
"browserslist": [
"chrome 110",
"firefox 111",
"safari 16.0"
],
"browserslist": ["chrome 110", "firefox 111", "safari 16.0"],
"resolutions": {
"optionator": "0.9.3",
"semver": "7.6.2"
Expand All @@ -213,4 +209,4 @@
"prismjs": "1.30.0"
}
}
}
}
Loading
0