Skip to content

feat: Enhance add-resource functionality in cli and add unit tests#323

Merged
MaojiaSheng merged 7 commits intovolcengine:mainfrom
shaoeric:main
Feb 27, 2026
Merged

feat: Enhance add-resource functionality in cli and add unit tests#323
MaojiaSheng merged 7 commits intovolcengine:mainfrom
shaoeric:main

Conversation

@shaoeric
Copy link
Contributor

@shaoeric shaoeric commented Feb 27, 2026

Description

support server-client cli(python & rust) add directory resource

Related Issue

discussion: #83

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test update

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this on the following platforms:
    • Linux
    • macOS
    • Windows

Checklist

  • My code follows the project's coding style
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

- Updated CLI commands to support additional parameters: --ignore-dirs, --include, --exclude, and --no-directly-upload-media.
- Modified resource processing to handle new parameters for directory scanning and file inclusion/exclusion.
- Improved README documentation for CLI usage and examples.

This update allows for more flexible resource management and better control over file uploads.
@shaoeric shaoeric marked this pull request as draft February 27, 2026 02:38
- Added support for additional parameters: --no-strict, --ignore-dirs, --include, --exclude, and --no-directly-upload-media.
- Updated the handling of resource addition to accommodate new options for improved directory scanning and file management.
- Enhanced README documentation with examples for the new functionality.

This update provides users with greater flexibility in managing resources.
@shaoeric shaoeric marked this pull request as ready for review February 27, 2026 03:20
@MaojiaSheng
Copy link
Collaborator

lgtm

@MaojiaSheng MaojiaSheng merged commit c85acbb into volcengine:main Feb 27, 2026
28 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenViking project Feb 27, 2026
BytedanceFu pushed a commit that referenced this pull request Feb 27, 2026
)

* tests(parsers): add unit tests for office extensions within add_resource directory

* feat: enhance add-resource functionality with new options

- Updated CLI commands to support additional parameters: --ignore-dirs, --include, --exclude, and --no-directly-upload-media.
- Modified resource processing to handle new parameters for directory scanning and file inclusion/exclusion.
- Improved README documentation for CLI usage and examples.

This update allows for more flexible resource management and better control over file uploads.

* feat(cli): enhance add-resource command with new options

- Added support for additional parameters: --no-strict, --ignore-dirs, --include, --exclude, and --no-directly-upload-media.
- Updated the handling of resource addition to accommodate new options for improved directory scanning and file management.
- Enhanced README documentation with examples for the new functionality.

This update provides users with greater flexibility in managing resources.

* chore(cli): remove unused cargo feature for edition 2024 in Cargo.toml

* chore(pyproject): update tag_regex in setuptools_scm configuration for version parsing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants