Skip to content

Conversation

@vadimturkov
Copy link
Collaborator

Closing #797.

@ePaul
Copy link
Member

ePaul commented Apr 30, 2024

👍

Copy link
Member

@tfrauenstein tfrauenstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx a lot for the proposal. LGTM -- provided editorial change proposals.

vadimturkov and others added 6 commits May 2, 2024 10:01
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
@vadimturkov
Copy link
Collaborator Author

@tfrauenstein thanks for the suggestions. Applied.

vadimturkov and others added 2 commits May 2, 2024 10:30
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
Co-authored-by: Thomas Frauenstein <thomas.frauenstein@zalando.de>
@tfrauenstein
Copy link
Member

👍

@vadimturkov
Copy link
Collaborator Author

@ePaul you marked this PR with zally-change label, does it mean we need to adjust Zally?

@ePaul
Copy link
Member

ePaul commented May 6, 2024

@vadimturkov Hmm, that would be the meaning of this label, but looking at the text again, I don't remember anymore why I did this.

@ePaul
Copy link
Member

ePaul commented May 6, 2024

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants