This repository was archived by the owner on Aug 15, 2025. It is now read-only.
CLI worker commands for files and file-contents#303
Closed
Nanashi-lab wants to merge 2 commits intogolemcloud:mainfrom
Closed
CLI worker commands for files and file-contents#303Nanashi-lab wants to merge 2 commits intogolemcloud:mainfrom
Nanashi-lab wants to merge 2 commits intogolemcloud:mainfrom
Conversation
Nanashi-lab
commented
Jul 29, 2025
| use cargo_toml::{ | ||
| Dependency, DependencyDetail, DepsSet, Edition, Inheritable, LtoSetting, Manifest, Profile, | ||
| Profiles, StripSetting, Workspace, | ||
| Profiles, StripSetting, Value, Workspace, |
Contributor
Author
There was a problem hiding this comment.
removed use toml::Value, in favour of cargo_toml:: Value, to fix this error.
cargo_toml = "0.22.1" (which depends on toml = "0.9.x")
toml = "0.8.19" (direct dependency)
Contributor
Author
|
@vigoo I am ready for a review, I updated this to latest, 1.3.0-dev-21, tested this with the new changes to Golem FileSystemNode. Thank you. |
Contributor
|
Will need to be rebased after merging my golem-cli PR; I'll try that before merging the repos to get this in if possible, if not will write there about reopening it in the other one. |
Contributor
|
Let's rather reopen it in the other repo |
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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
/closes golemcloud/golem#1922
/claim golemcloud/golem#1922
Video Link for usage Link
Add two new cli commands to worker, files and file-contents. These changes are independant of the PR in golem.
files
Arguments
<WORKER_NAME>and[PATH] - Path to the directory to list files from [default: /]Example (Success)
Example[Failure]
file-contents
Arguments
<WORKER_NAME>and<PATH>, optional--output, to specific output location and nameExample [Success]
golem-dev worker file-contents test:sc/test /in/baz.txt --output test.txt Selected profile: local Fuzzy matched pattern test:sc as test:sc Downloading file from worker test:sc/test at path /in/ File Saved to path test.txtExample [Failure]
golem-dev worker file-contents test:sc/test /in/ Selected profile: local Fuzzy matched pattern test:sc as test:sc Downloading file from worker test:sc/test at path /in/ Failed to download file from worker test:sc/test at path /in/: Worker Service - Error: 400 Bad Request, Bad file type: /in error: Worker Service - Error: 400 Bad Request, Bad file type: /in