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

Bring zipkin-finagle up to date #149

Merged
merged 5 commits into from
Jun 22, 2024
Merged

Bring zipkin-finagle up to date #149

merged 5 commits into from
Jun 22, 2024

Conversation

reta
Copy link
Contributor

@reta reta commented Apr 20, 2024

Bring zipkin-finagle up to date

@reta
Copy link
Contributor Author

reta commented Jun 20, 2024

@codefromthecrypt would really appreciate if you could find a time to look at it: this is first step to bring the project back to shape, the other things (Docker, Github Actions, SPDX licenses, ... ) to follow shortly, thanks!

@reta reta force-pushed the issue-142 branch 2 times, most recently from 4fba2ec to 0ac1a36 Compare June 20, 2024 01:48
Signed-off-by: Andriy Redko <drreta@gmail.com>
Copy link
Member

@codefromthecrypt codefromthecrypt left a comment

Choose a reason for hiding this comment

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

looks good, but maybe revert the move to scala 2.13 unless finagle is only 2.13 now.. in any case look at build-bin/deploy for what I mean

.github/workflows/test.yml Outdated Show resolved Hide resolved
core/pom.xml Outdated
@@ -19,12 +19,12 @@

<parent>
<groupId>io.zipkin.finagle2</groupId>
<artifactId>zipkin-finagle-parent_2.12</artifactId>
<version>22.4.0</version>
<artifactId>zipkin-finagle-parent_2.13</artifactId>
Copy link
Member

Choose a reason for hiding this comment

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

hmm I thought in our build we did some tricks to dual publish to 12 and 13 via replacement?

@reta
Copy link
Contributor Author

reta commented Jun 20, 2024

looks good, but maybe revert the move to scala 2.13 unless finagle is only 2.13 now.. in any case look at build-bin/deploy for what I mean

It is both, you are very right, I will take care of that, thanks!

reta added 2 commits June 21, 2024 18:00
Signed-off-by: Andriy Redko <drreta@gmail.com>
@codefromthecrypt
Copy link
Member

another thing I think we should figure out how to turn off scala-steward and close all the unaddressed spam PRs..

@reta
Copy link
Contributor Author

reta commented Jun 22, 2024

another thing I think we should figure out how to turn off scala-steward and close all the unaddressed spam PRs..

Sure, I think once we merge this one, many will be auto closed

reta and others added 2 commits June 22, 2024 08:16
Co-authored-by: Adrian Cole <64215+codefromthecrypt@users.noreply.github.com>
Signed-off-by: Andriy Redko <drreta@gmail.com>
@reta reta merged commit 1e85177 into openzipkin:master Jun 22, 2024
1 check passed
@codefromthecrypt
Copy link
Member

Thanks @reta!

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