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

Remove packages.config from UI project #8950

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

Conversation

ivandrofly
Copy link
Member

Remove packages.config as the project is already using PackageReference

The packages.config file was removed to potentially streamline package management and reduce redundancy. This change is likely in preparation for using PackageReference or another method for handling dependencies more efficiently.

Signed-off-by: Ivandro Jao <Ivandrofly@gmail.com>
@ivandrofly
Copy link
Member Author

For some reason appveyor is still looking for src/ui/packages.config....

…age restore

In light of recent changes in software development practices, we have migrated the build and test scripts in appveyor.yml from MSBuild to dotnet. The "msbuild" command has been replaced with "dotnet build" and "vstest.console" with "dotnet test". Additionally, the environment variable for NuGet package restore has been set to false in both appveyor.yml and SubtitleEdit.csproj files. By not restoring packages on each build, we reduce network traffic and improve build performance.
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.

1 participant