saturdaymp / Migrator

Migrate database changes.
https://saturdaymp.com
MIT License
1 stars 0 forks source link

Sweep: Create GitHub Action to compile and run the unit tests #8

Closed sweep-ai[bot] closed 1 week ago

sweep-ai[bot] commented 5 months ago

Description

This pull request updates the App.config file within the MigratorTests project to reflect changes in the database connection string format and assembly binding redirects.

Summary

Fixes #7.


💡 To get Sweep to edit this pull request, you can:

This is an automated message generated by Sweep AI.

sweep-ai[bot] commented 5 months ago

Rollback Files For Sweep

This is an automated message generated by Sweep AI.

sweep-ai[bot] commented 5 months ago

Sweep: PR Review

Source/MigratorTests/App.config

The connection string for the "Master" database has been modified by changing the attributes "Uid" and "Pwd" to "User Id" and "Password".

Sweep Found These Issues

  • The change from "Uid" and "Pwd" to "User Id" and "Password" in the connection string may cause connection issues if the database connection library does not recognize these new attributes.
  • https://github.com/saturdaymp/Migrator/blob/fced949970f9623a3f54a7a2e84b28fa23fa5e4a/Source%2FMigratorTests%2FApp.config#L4 [View Diff](https://github.com/saturdaymp/Migrator/pull/8/files#diff-036adf66c426c9f25d7dc0dae81ddd7c6dea789d8d4f827dad23843c4291221eR4)