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

fix!: copy contents of directory instead of the directory if store is used for directories #2326

Merged
merged 9 commits into from Apr 29, 2024

Conversation

h4ck3rk3y
Copy link
Contributor

Closes #2227

@h4ck3rk3y h4ck3rk3y added the DO NOT MERGE DO NOT MERGE THIS PR label Mar 26, 2024
@barnabasbusa
Copy link
Contributor

Before this gets merged we have to have a PR ready to go for all major kurtosis packages, and all CI jobs have to be tested with this new version, as I expect it to break a lot of things!

@h4ck3rk3y h4ck3rk3y added DO NOT MERGE DO NOT MERGE THIS PR and removed DO NOT MERGE DO NOT MERGE THIS PR labels Apr 29, 2024
@h4ck3rk3y h4ck3rk3y added this pull request to the merge queue Apr 29, 2024
Merged via the queue into main with commit 4c776be Apr 29, 2024
55 of 57 checks passed
@h4ck3rk3y h4ck3rk3y deleted the gyani/store branch April 29, 2024 10:55
github-merge-queue bot pushed a commit that referenced this pull request Apr 29, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.89.0](0.88.19...0.89.0)
(2024-04-29)


### ⚠ BREAKING CHANGES

* copy contents of directory instead of the directory if store is used
for directories
([#2326](#2326))

### Bug Fixes

* copy contents of directory instead of the directory if store is used
for directories
([#2326](#2326))
([4c776be](4c776be)),
closes [#2227](#2227)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
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.

plan/run_sh/store: providing a directory path doesn't copy the content of it
3 participants