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

Common: add toThrift and toRaw methods to CollectorPayload (close #345) #364

Merged
merged 6 commits into from
Oct 6, 2020

Conversation

benjben
Copy link
Contributor

@benjben benjben commented Oct 2, 2020

No description provided.

Copy link
Contributor

@chuwy chuwy left a comment

Choose a reason for hiding this comment

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

Looks awesome!

@chuwy
Copy link
Contributor

chuwy commented Oct 6, 2020

Hey @benjben! Do you mind me adding #358 and #357 to this PR? They're also auxuliary commits without any impact on the assets. Just to make #346 PR clean.

@benjben
Copy link
Contributor Author

benjben commented Oct 6, 2020

Sure @chuwy !

@chuwy chuwy mentioned this pull request Oct 6, 2020
12 tasks
@chuwy
Copy link
Contributor

chuwy commented Oct 6, 2020

Added those and couple of minor bumps to make Snyk completely happy. @oguzhanunlu could you have a look?

@oguzhanunlu
Copy link
Member

It seems CI isn't happy @chuwy , is that expected?

@chuwy
Copy link
Contributor

chuwy commented Oct 6, 2020

No, not expected - you're right. I added this check in #357 because it can break publishing. Will fix it now.

@chuwy
Copy link
Contributor

chuwy commented Oct 6, 2020

Green now, @oguzhanunlu (no idea what those -0.02%).

Copy link
Member

@oguzhanunlu oguzhanunlu left a comment

Choose a reason for hiding this comment

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

LGTM!

@chuwy chuwy merged commit 96206e2 into develop Oct 6, 2020
@chuwy chuwy deleted the feature/collectorPayload branch October 21, 2020 14:54
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

3 participants