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

Adding Support for .Net 9 #919

Open
wants to merge 35 commits into
base: main
Choose a base branch
from
Open

Adding Support for .Net 9 #919

wants to merge 35 commits into from

Conversation

TalZaccai
Copy link
Contributor

No description provided.

@TalZaccai TalZaccai linked an issue Jan 17, 2025 that may be closed by this pull request
Request to add support for .Net 9 #885
Open
TalZaccai and others added 21 commits January 17, 2025 16:39
@TalZaccai
some fixes
8059a3a
@TalZaccai
Merge branch 'main' into tedhar/net9
80b3dc4
@darrenge
Added .net90 to CI and the Nightly GitHub Action
84cc8cd
@darrenge
Fixed Garnet Worker to be both .net80 and .net90 and updated ADO pipe…
d0d948a 
…lines to use net9.0
@darrenge
Updated Tsav CI that runs in ADO when mirrored
121d14f
@TalZaccai
Fix to RedisCallErrors
faae98a
@TalZaccai
test
2b67210
@TalZaccai
fix
3e3ed56
@TalZaccai
Skipping RedisCallErrors test
d94a323
@darrenge
Merge branch 'tedhar/net9' of https://github.com/microsoft/garnet int…
b49a1b5 
…o tedhar/net9
@TalZaccai
Reverting ignored test
a74e664
@TalZaccai
Updating CI
0214669
@TalZaccai
CI fix
ebd5f05
@TalZaccai
Ignoring RedisCallErrors test when environment variable is not set
3fc3ffe
@darrenge
Added net90 to the external release pipeline. Required changed to pub…
027fe0e 
…lish profile. GitHub release section working now. Need to finish Nuget. The actual GH Release task and Nuget Push are disabled at this point.
@darrenge
Merge branch 'tedhar/net9' of https://github.com/microsoft/garnet int…
99621f7 
…o tedhar/net9
@darrenge
Fixed bug in compressed file directory
0d668ec
@darrenge
Another try on fixing the destination directory issue
2cedd73
@darrenge
Added CodeQL.yml which is custom so it works with net90 as well. Upda…
c82efda 
…ted GarnetHost to get all the runtimes
@TalZaccai
merging with latest main
064ac3f
@darrenge
Set RunTImes to only be packed once since they are C++ and not depend…
37e1cbd 
…ent on framework
@darrenge darrenge requested a review from badrishc January 30, 2025 22:40
@darrenge darrenge requested a review from mgravell January 30, 2025 23:15
@darrenge darrenge marked this pull request as ready for review January 30, 2025 23:24
@darrenge darrenge self-requested a review as a code owner January 30, 2025 23:24
@darrenge darrenge removed their request for review January 30, 2025 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PaulusParssinen PaulusParssinen PaulusParssinen left review comments

@badrishc badrishc Awaiting requested review from badrishc

@mgravell mgravell Awaiting requested review from mgravell

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.

Request to add support for .Net 9
4 participants
@TalZaccai @PaulusParssinen @TedHartMS @darrenge