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

feat: Compute content hash when compressing files artifact #1041

Merged
merged 2 commits into from Aug 3, 2023

Conversation

gbouv
Copy link
Contributor

@gbouv gbouv commented Aug 2, 2023

Description:

Effectively this does nothing right now. It will allow us to cleverly update files artifact based on their hash in a future PR

Is this change user facing?

NO

References (if applicable):

@gbouv gbouv force-pushed the gbouv/compressed-file-hash branch from f3ac306 to 30aabc4 Compare August 2, 2023 10:57
@gbouv gbouv requested a review from Peeeekay August 2, 2023 11:14
@gbouv gbouv self-assigned this Aug 2, 2023
@gbouv gbouv enabled auto-merge (squash) August 3, 2023 09:13
@gbouv gbouv merged commit 510ffe2 into main Aug 3, 2023
29 checks passed
@gbouv gbouv deleted the gbouv/compressed-file-hash branch August 3, 2023 09:30
victorcolombo pushed a commit that referenced this pull request Aug 3, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.81.2](0.81.1...0.81.2)
(2023-08-03)


### Features

* Compute content hash when compressing files artifact
([#1041](#1041))
([510ffe2](510ffe2))


### Bug Fixes

* Fix comment about sidecar
([#1053](#1053))
([d9b07ea](d9b07ea))
* Use the local grpc-file-transfer library version
([#1056](#1056))
([59fa980](59fa980))

---
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.

None yet

2 participants