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

v0: Fixes Cake Build and Upgrades Dependencies #1040

Closed
wants to merge 5 commits into from

Conversation

Jaben
Copy link
Contributor

@Jaben Jaben commented Aug 22, 2024

  • Updated MongoDB to 2.28.
  • Updated newtonsoft.json to 13.0.3
  • Updated Npgsql to 4.1.14
  • Updated Owin to 4.2.2
  • Updated System.Data.SqlClient to 4.8.6

I updated the cake build, so hopefully that works again.

@Jaben Jaben changed the base branch from develop-v1 to develop-v0 August 22, 2024 22:04
@Jaben
Copy link
Contributor Author

Jaben commented Aug 22, 2024

@rasmus this should fix v0 branch builds.

@Jaben
Copy link
Contributor Author

Jaben commented Sep 4, 2024

This is a little broken in that the packed Nugets will target the latest EventFlow -- not v0.83.

@rasmus
Copy link
Member

rasmus commented Oct 18, 2024

@Jaben what's the recommendation here? I haven't touched the v0 code base in a long while

@Jaben
Copy link
Contributor Author

Jaben commented Oct 18, 2024

@Jaben what's the recommendation here? I haven't touched the v0 code base in a long while

A new version of "EventFlow v0" should be released -- and then I'd add a hard reference to that new build version of the v0 branch:

image

@Jaben
Copy link
Contributor Author

Jaben commented Oct 18, 2024

I'll go ahead and update this PR to support.

@rasmus
Copy link
Member

rasmus commented Oct 19, 2024

I'm working on getting a GHA runner up and running for EventFlow that has Docker properly configured (Linux containers).

@rasmus
Copy link
Member

rasmus commented Oct 20, 2024

Thanks for this ❤️ it really help to get it up and v0 running again. Closing this as I tried to replicate the minimum needed in #1054 and that is now merged. It also remove Cake

@rasmus rasmus closed this Oct 20, 2024
@Jaben
Copy link
Contributor Author

Jaben commented Oct 21, 2024

Thanks for this ❤️ it really help to get it up and v0 running again. Closing this as I tried to replicate the minimum needed in #1054 and that is now merged. It also remove Cake

Absolutely! Thank you!

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.

2 participants