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

Initial working #11

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

gep13
Copy link
Member

@gep13 gep13 commented Feb 10, 2020

No description provided.

@gep13 gep13 requested a review from AdmiringWorm February 10, 2020 22:04
@AdmiringWorm
Copy link
Member

We won't be able to test these changes on appveyor before the target branch have updated the appveyor image used to "Visual Studio 2019" (appveyor don't use the appveyor.yml file in a pr, only the one on the target branch).

@AdmiringWorm
Copy link
Member

I re-implemented the nuget package creation to better conform to creating packages for multiple runtimes (since we can use the same nuspec file for each runtime).

gep13 and others added 3 commits February 11, 2020 19:36
This allows us to use a single nuspec file for creating all
of the supported runtimes
@gep13 gep13 force-pushed the runtime-generation branch from e3895c9 to 6897980 Compare February 11, 2020 19:36
@gep13
Copy link
Member Author

gep13 commented Feb 11, 2020

@AdmiringWorm thank you very much for taking a look at this! The refactoring that you have done is a stroke of genius! I have rebased this branch on the latest develop, which is now running on the right AppVeyor base image, and the build is now working. There is still a little bit of re-work to be done here, but basically it seems to be working as expected.

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.

2 participants