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.

Sign up for GitHub

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

Jump to bottom

chore: upgrade dependencies #47

Merged
merged 5 commits into from
Mar 3, 2021
Merged

chore: upgrade dependencies #47

merged 5 commits into from
Mar 3, 2021

Conversation

valdo404
Copy link
Contributor

  • Upgrade scalapb
  • Upgrade circe

@StephaneBour StephaneBour requested a review from malves February 26, 2021 14:29
@valdo404 valdo404 force-pushed the upgrade-dependencies branch 3 times, most recently from 8573af0 to b52994f Compare February 26, 2021 21:29
@valdo404 valdo404 force-pushed the upgrade-dependencies branch from b52994f to 5eab0c8 Compare March 1, 2021 09:37
@valdo404 valdo404 force-pushed the upgrade-dependencies branch from 5eab0c8 to afc54fe Compare March 1, 2021 09:42
gontarzpawel
gontarzpawel approved these changes Mar 2, 2021
View reviewed changes
Copy link

@gontarzpawel gontarzpawel left a comment

Choose a reason for hiding this comment

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

Revamp of scala-openrtb?! LGTM

@valdo404
Copy link
Contributor Author

valdo404 commented Mar 2, 2021

Revamp of scala-openrtb?! LGTM

Man it's just an upgrade of dependencies. ;) but I think there is more to come.

@malves malves merged commit 222b986 into Powerspace:master Mar 3, 2021
@valdo404 valdo404 deleted the upgrade-dependencies branch March 3, 2021 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StephaneBour StephaneBour StephaneBour approved these changes

@gontarzpawel gontarzpawel gontarzpawel approved these changes

@rlebran rlebran rlebran approved these changes

@mikaelBrosset mikaelBrosset mikaelBrosset approved these changes

@emanuele-pirro emanuele-pirro emanuele-pirro approved these changes

@sithis2a sithis2a sithis2a approved these changes

@malves malves malves approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@valdo404 @malves @sithis2a @emanuele-pirro @mikaelBrosset @rlebran @gontarzpawel @StephaneBour