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

Switching to managed code version of functionsnethost #1704

Merged
merged 18 commits into from
Jul 26, 2023

Conversation

kshyju
Copy link
Member

@kshyju kshyju commented Jun 30, 2023

Pulling in changes from feature branch to main.

The PR branch was created from the feature branch and then rebased on top of main. Only conflicts were the in the DotnetWorker.csproj and DotnetWorker.Grpc.csproj, I took the feature branch version during rebase. Updated these 2 files later to reflect the new version numbers we want.

I may create another PR just to adjust the version numbers of the packages when we are ready to release.

@kshyju kshyju requested review from fabiocav and brettsam and removed request for fabiocav June 30, 2023 01:03
@kshyju
Copy link
Member Author

kshyju commented Jul 5, 2023

I may create another PR just to adjust the version numbers of the packages when we are ready to release.

First iteration of Azure Functions Custom .NET Host(Validated in Windows)
* Adding support for linux cold start.

* Using native "get_hostfxr_path" method from "nethost". Removed PathResolver code (which was our version to do the same thing)

* Update build YAML to copy all  files instead of just .exe.

* Linted yaml file. fixed the copy step to copy only needed dependencies

* YAML

* Get the NativeApplication instance from pointer and call the methods instead of static method call.

* Revert "Get the NativeApplication instance from pointer and call the methods instead of static method call."

This reverts commit 2f54a1c.

* Using NET8 Preview4.

* NET6 support on Debian

* Version bump for DotNetIsolatedNativeHost package

* Add net6.0;net7.0; to TargetFrameworks.

* Removing the shim for Windows.
@kshyju kshyju force-pushed the shkr/rebase_on_main-feature-functionsnethost-managedcode branch from 4243ff4 to 4d7cd80 Compare July 11, 2023 18:16
@kshyju
Copy link
Member Author

kshyju commented Jul 11, 2023

Rebased on main again this morning.

@kshyju kshyju requested a review from fabiocav July 20, 2023 23:51
@kshyju kshyju merged commit 2d16152 into main Jul 26, 2023
28 checks passed
@kshyju kshyju deleted the shkr/rebase_on_main-feature-functionsnethost-managedcode branch July 26, 2023 21:12
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.

4 participants