ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
16 stars 59 forks source link

chore: fix knip commands by removing format prefix #933

Closed gitcoindev closed 1 month ago

gitcoindev commented 1 month ago

Fix for incorrect knip commands in package.json.

github-actions[bot] commented 1 month ago

Unused files (48)

src/main.ts, src/bindings/event.ts, src/handlers/processors.ts, src/helpers/contracts.ts, src/helpers/label.ts, src/helpers/payout.ts, src/tests/after-all-handler.ts, src/tests/before-all-handler.ts, src/tests/commands-test.ts, src/tests/test-repo-config.ts, src/tests/test-suite.ts, src/tests/utils.ts, src/types/commit.ts, src/types/github-events.ts, src/utils/check-github-rate-limit.ts, src/utils/generate-configuration.ts, src/adapters/github/github-actions.ts, src/handlers/access/labels-access.ts, src/handlers/comment/comment-created.ts, src/handlers/label/label.ts, src/handlers/penalty/add-penalty.ts, src/handlers/pricing/pricing-label.ts, src/handlers/pricing/sync-labels-to-config.ts, src/handlers/pull-request/create-devpool-pr.ts, src/handlers/push/check-modified-base-rate.ts, src/handlers/push/push.ts, src/handlers/push/update-base-rate.ts, src/handlers/shared/pricing.ts, src/handlers/comment/handlers/first.ts, src/handlers/comment/handlers/issue-closed.ts, src/handlers/wildcard/unassign/aggregate-assignee-activity.ts, src/handlers/wildcard/unassign/assign-event-found.ts, src/handlers/wildcard/unassign/check-if-follow-up-already-posted.ts, src/handlers/wildcard/unassign/check-task-to-unassign.ts, src/handlers/wildcard/unassign/check-tasks-to-unassign.ts, src/handlers/wildcard/unassign/disqualify-idle-assignees.ts, src/handlers/wildcard/unassign/get-active-assignees-in-disqualify-duration.ts, src/handlers/wildcard/unassign/get-active-assignees-in-follow-up-duration.ts, src/handlers/wildcard/unassign/get-active-assignees.ts, src/handlers/wildcard/unassign/get-all-commits-from-pull-request.ts, src/handlers/wildcard/unassign/get-all-events.ts, src/handlers/wildcard/unassign/is-correct-type.ts, src/handlers/wildcard/unassign/remind-non-eliminated-assignees.ts, src/handlers/wildcard/unassign/unassign.ts, src/handlers/comment/handlers/delegate-compute/delegated-compute.ts, src/handlers/comment/handlers/delegate-compute/remote-permissions.ts, src/handlers/comment/handlers/issue/get-collaborator-ids-for-repo.ts, src/handlers/comment/handlers/issue/get-pull-request-comments.ts

Unused dependencies (17)

Filename dependencies
package.json @commitlint/clibr/>`@netlify/functions`<br/@octokit/restbr/>`@openzeppelin/contracts`<br/@probot/adapter-github-actionsbr/>`@typescript-eslint/typescript-estree`<br/@uniswap/permit2-sdkbr/>`@vercel/ncc`<br/js-tiktoken
jsdom
libsodium-wrappers
lint-staged
markdown-it
smee-client
tsx
yaml
zlib

Unused devDependencies (7)

Filename devDependencies
package.json @types/jestbr/>`@types/jsdom`<br/@types/libsodium-wrappersbr/>`@types/markdown-it`<br/@types/source-map-support
octokit
source-map-support

Unused exports (9)

Filename exports
src/helpers/issue.ts addLabelToIssue
addCommentToIssue
src/helpers/shared.ts shouldSkip
src/types/configuration-types.ts validateBotConfig
src/handlers/assign/assign-command-handler.ts assignCommandHandler
src/handlers/assign/check-pull-requests.ts checkPullRequests
src/handlers/pricing/handle-parent-issue.ts handleParentIssue
sortLabelsByValue
src/handlers/comment/handlers/comment-handler-main.ts commentParser

Unused types (7)

Filename types
src/types/handlers.ts WildCardHandler
Handler
src/types/payload.ts GitHubEvent
GitHubCommitsPayload
GitHubAssignEvent
GitHubPushPayload
GithubContent

Unused Enum Members (1)

Filename Enum Member
src/types/payload.ts UserType Organization

Unused Class Members (10)

Filename Class Member
src/adapters/supabase/helpers/tables/label.ts Label saveLabelChange
src/adapters/supabase/helpers/tables/locations.ts Locations locationResponse
user_id
comment_id
issue_id
repository_id
node_id
node_type
src/adapters/supabase/helpers/tables/settlement.ts Settlement addDebit
addCredit
gentlementlegen commented 1 month ago

@gitcoindev Thank you. Knip results are a disaster haha

Also realized that this is merged against main. Are we able to deploy the main branch as the current production Ubiquibot? Afaik this is still not the branch being deployed.

gitcoindev commented 1 month ago

@gitcoindev Thank you. Knip results are a disaster haha

Also realized that this is merged against main. Are we able to deploy the main branch as the current production Ubiquibot? Afaik this is still not the branch being deployed.

I have just seen your question and I am not sure about the answer. @0x4007 could you please confirm that this change should be merged into main branch from development branch after merging this pr or a separate pull request to main should be opened?

0x4007 commented 1 month ago

@gitcoindev Thank you. Knip results are a disaster haha Also realized that this is merged against main. Are we able to deploy the main branch as the current production Ubiquibot? Afaik this is still not the branch being deployed.

I have just seen your question and I am not sure about the answer. @0x4007 could you please confirm that this change should be merged into main branch from development branch after merging this pr or a separate pull request to main should be opened?

Production is actually running off of move-to-delegated-compute