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.

Sign up for GitHub

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

Jump to bottom

Add rapids-upload-artifacts-dir utility #43

Merged
merged 7 commits into from
Feb 18, 2023
Merged

Add rapids-upload-artifacts-dir utility #43

merged 7 commits into from
Feb 18, 2023

Commits on Feb 15, 2023

  1. Add rapids-upload-artifacts-dir utility

    @davidwendt
    davidwendt committed Feb 15, 2023
    Configuration menu
    Copy the full SHA
    7703143 View commit details
    Browse the repository at this point in the history

  2. fix permissions

    @davidwendt
    davidwendt committed Feb 15, 2023
    Configuration menu
    Copy the full SHA
    92d7175 View commit details
    Browse the repository at this point in the history

  3. fix typo in script

    @davidwendt
    davidwendt committed Feb 15, 2023
    Configuration menu
    Copy the full SHA
    1185544 View commit details
    Browse the repository at this point in the history

  4. add more double-quotes

    @davidwendt
    davidwendt committed Feb 15, 2023
    Configuration menu
    Copy the full SHA
    516df57 View commit details
    Browse the repository at this point in the history

  5. try more double quotes

    @davidwendt
    davidwendt committed Feb 15, 2023
    Configuration menu
    Copy the full SHA
    37e899e View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2023

  1. ensure loop works for empty dirs 

    `-e` ensures that the file/dir in the current iteration exists on the filesystem.

this is to account for situations where the directory is empty.

adapted from this post: https://unix.stackexchange.com/q/239772
    @ajschmidt8
    ajschmidt8 committed Feb 18, 2023
    Configuration menu
    Copy the full SHA
    3f73d29 View commit details
    Browse the repository at this point in the history

  2. unrelated exit code change 

    makes the exit code `0` for local uploads attempted with `rapids-upload-conda-to-s3`.

this makes it match `rapids-upload-artifacts-dir` and is probably the more appropriate setting.
    @ajschmidt8
    ajschmidt8 committed Feb 18, 2023
    Configuration menu
    Copy the full SHA
    07f00cc View commit details
    Browse the repository at this point in the history