Skip to content
Merged
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
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 20 additions & 0 deletions .github/workflows/npm-publish.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
name: Npm publish

on:
release:
types: [published]

jobs:
publish:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 12
registry-url: https://registry.npmjs.org/
- run: npm install
- run: npm run build
- run: npm publish
env:
NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -25,3 +25,5 @@ coverage/
.nyc_output/
.idea
yarn.lock

.claude/settings.local.json
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This ignores only settings.local.json inside .claude/. Claude Code tooling may generate other files in that directory (e.g. settings.json, memory files, task lists). Consider broadening the pattern to avoid accidentally committing those in the future:

Suggested change
.claude/settings.local.json
.claude/

If there are files under .claude/ that should be tracked (e.g. project-level CLAUDE.md or shared settings), you can unignore them explicitly with !.claude/CLAUDE.md.

3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
## CHANGE LOG

## 7.15.2
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Please add a blank line between the ## CHANGE LOG header and the new version entry ## 7.15.2 to maintain consistent formatting and improve readability.

Suggested change
## 7.15.2
## 7.15.2

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no blank line between the top-level ## CHANGE LOG heading and the new ## 7.15.2 section. Every prior version entry is separated by a blank line, so this breaks the established formatting convention.

Suggested change
## 7.15.2
## 7.15.2

- 对象存储,修复 pfop pipeline 参数无效问题
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The changelog entry mentions a fix for the pfop pipeline parameter, but the actual code changes are missing from this PR. The current implementation of pfop in qiniu/fop.js does not correctly handle optional arguments (e.g., when pipeline is omitted and options is passed as the 4th argument), which leads to the pipeline parameter being set to an object and the callback being lost. Please include the fix in this pull request.


## 7.15.1
- 对象存储,开放分片上传部分接口

Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "qiniu",
"version": "7.15.1",
"version": "7.15.2",
"description": "Node wrapper for Qiniu Resource (Cloud) Storage API",
"main": "index.js",
"directories": {
Expand Down
Loading