Skip to content
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

fetch #16

Merged
merged 3 commits into from
Sep 2, 2024
Merged

fetch #16

merged 3 commits into from
Sep 2, 2024

Conversation

thangckt
Copy link
Owner

@thangckt thangckt commented Sep 2, 2024

No description provided.

pre-commit-ci bot and others added 3 commits August 28, 2024 16:29
<!--pre-commit.ci start-->
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.1 →
v0.6.2](astral-sh/ruff-pre-commit@v0.6.1...v0.6.2)
<!--pre-commit.ci end-->

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Fix #486.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit


- **New Features**
- Updated documentation to clarify file handling with the `Local`
context type, emphasizing the use of symlinks for improved efficiency.
- Introduced a new class attribute for configurable symlink options
during file operations.
	- Added a new key for symlink configuration in the remote profile JSON.

- **Bug Fixes**
- Enhanced error handling for file copying processes to ensure smoother
operation.

- **Tests**
- Added a new test class for local context submissions, improving test
coverage for file handling scenarios.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Documentation**
- Updated documentation to clarify the handling of local directory
accessibility in the `Local` job execution context, ensuring users
understand to disable symlink options when local directories are not
accessible.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@thangckt thangckt merged commit 1ded55e into thangckt:master Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants