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

Updated C# projects to CPS/SDK-style format #161

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

icnocop
Copy link
Collaborator

@icnocop icnocop commented Sep 22, 2023

Converted packages.config to package references
Removed unused using statements
Replaced local copy of StyleCop and NUnit with NuGet packages Using MSBuild to restore nuget packages instead of nuget.exe

Converted packages.config to package references
Removed unused using statements
Replaced local copy of StyleCop and NUnit with NuGet packages
Using MSBuild to restore nuget packages instead of nuget.exe
@icnocop icnocop requested a review from dblock September 22, 2023 01:34
@icnocop
Copy link
Collaborator Author

icnocop commented Sep 22, 2023

I've been experiencing random crashes from htmlInstaller.exe, especially during test runs on the build agents.

The crash dumps, attached within build artifacts, indicate a stack trace originating from htmllayout.dll.

I've been trying to migrate from HtmlLayout to Sciter.JS but that is still work in progress (and not part of this PR).

As a work-around, re-running the build may not cause a crash.

You can see the same code builds and tests pass successfully:
https://github.com/icnocop/dotnetinstaller/actions/runs/6268834204

@icnocop icnocop added the chore label Nov 23, 2023
@icnocop
Copy link
Collaborator Author

icnocop commented Jan 15, 2024

I would like this PR merged before working on any new issues, like #163 for example.

@dblock dblock merged commit b73dccb into dotnetinstaller:master Jan 16, 2024
2 checks passed
@dblock
Copy link
Collaborator

dblock commented Jan 16, 2024

Looks good, did a cursory look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants