-
Notifications
You must be signed in to change notification settings - Fork 12
feat: v-next #42
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
Open
ComfortablyCoding
wants to merge
32
commits into
master
Choose a base branch
from
v-next
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
feat: v-next #42
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
c87c8e0
refactor: cleanup and move to esm
ComfortablyCoding fbcded4
chore: update deps, configs and tests
ComfortablyCoding a58df6f
fix(workflows): actions are not under `.github`
ComfortablyCoding 71118ee
fix(setup action): pnpm version is not specified
ComfortablyCoding ceefbca
chore: update ignore files
ComfortablyCoding 74ea6c5
chore(jsdoc): add a link corresponding API doc for functions
ComfortablyCoding 6f3a5f6
chore(route): update typing for client
ComfortablyCoding 53c0d5b
fix(comment): add missing threaded endpoints
ComfortablyCoding 36dbc25
chore(setToken): add additional space to jsdoc for consistency
ComfortablyCoding 29034a3
Merge branch 'v-next' of https://github.com/ComfortablyCoding/clickup…
ComfortablyCoding b0d1d9d
docs: remove manual docs
ComfortablyCoding a23d989
doc: add auto generated API
ComfortablyCoding 39ca28e
unify naming and add missing routes
ComfortablyCoding a1dd72c
add version support
ComfortablyCoding 939ec00
remove index route import
ComfortablyCoding 1c30b9e
remove version fro prefixUrl
ComfortablyCoding 6986475
fix token not being set with setToken
ComfortablyCoding b24ac67
auto snakeCase body keys
ComfortablyCoding 763e2f6
Rename params to options and utilize query over params
ComfortablyCoding 7cd44f9
export routes under default
ComfortablyCoding 4bc2bec
remove leading route slash
ComfortablyCoding 8a68622
move to ofetch, add rate limiting and snake_case keys
ComfortablyCoding 028a926
add tests
ComfortablyCoding 4b7d286
move route top level
ComfortablyCoding c80c4be
add docs
ComfortablyCoding 3043724
remove pnpm-workspace
ComfortablyCoding 3cf9586
update prettier rule
ComfortablyCoding 7416e69
add required version prefix to auth routes
ComfortablyCoding 1fccb26
fix version not being passed to routes
ComfortablyCoding 59210bf
fix token not acknowledged
ComfortablyCoding a7485fe
improve error format
ComfortablyCoding d6a3915
add jsdoc for createClickup
ComfortablyCoding File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| name: Setup | ||
| description: Install dependencies and build package | ||
|
|
||
| runs: | ||
| using: composite | ||
| steps: | ||
| - name: Install Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: "lts/*" | ||
| check-latest: true | ||
| registry-url: https://registry.npmjs.org/ | ||
|
|
||
| - uses: pnpm/action-setup@v3 | ||
| name: Install pnpm | ||
| with: | ||
| run_install: false | ||
| version: 9 | ||
|
|
||
| - name: Install dependencies | ||
| shell: bash | ||
| run: pnpm install |
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,20 +1,18 @@ | ||
| name: PR Test | ||
| name: Test PR | ||
|
|
||
| on: [pull_request] | ||
|
|
||
| jobs: | ||
| test-pr: | ||
| test: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout branch | ||
| uses: actions/checkout@v2 | ||
|
|
||
| - name: Install Node v16 | ||
| uses: actions/setup-node@v2 | ||
| - name: checkout | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| node-version: '16.x' | ||
| registry-url: 'https://registry.npmjs.org' | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Clean install deps | ||
| run: npm ci | ||
| - name: setup | ||
| uses: ./.github/actions/setup | ||
|
|
||
| - name: Run tests | ||
| run: npm test | ||
| - name: test | ||
| run: pnpm test -- -- --changed origin/${{ github.base_ref }} --passWithNoTests |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| name: Publish | ||
|
|
||
| on: | ||
| release: | ||
| types: [published] | ||
|
|
||
| jobs: | ||
| publish-npm: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: checkout | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: setup | ||
| uses: ./.github/actions/setup | ||
|
|
||
| - name: publish to npm | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
| NPM_CONFIG_PROVENANCE: true | ||
| run: | | ||
| pnpm publish \ | ||
| --access=public \ | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| export default { | ||
| $schema: "https://json.schemastore.org/prettierrc", | ||
| tabWidth: 2, | ||
| semi: true, | ||
| singleQuote: false, | ||
| useTabs: true, | ||
| printWidth: 120, | ||
| }; |
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Copilot Autofix
AI 10 days ago
To resolve the issue, add a
permissionsblock to the workflow configuration to restrict theGITHUB_TOKENto the least necessary privileges. The best location is at the root of the workflow (under the workflow name or triggers) if you want to define permissions for all jobs, or within each job's config if you need more granularity.For this workflow, a minimal permissions block such as
contents: readwill almost always suffice for publishing to npm (actual publishing is authenticated via theNODE_AUTH_TOKEN), though if you use npm provenance you may also wantid-token: write. To strictly follow the principle of least privilege for the visible steps, add the following at the top level just after the workflow name and before theon:trigger:Edit
.github/workflows/publish.ymlto insert this block aftername: Publish. No new packages or external imports are required.