Skip to content

fix: align sample project and assembly names#174

Open
alafleur-genetec wants to merge 1 commit into
mainfrom
fix/copy-configuration-typo
Open

fix: align sample project and assembly names#174
alafleur-genetec wants to merge 1 commit into
mainfrom
fix/copy-configuration-typo

Conversation

@alafleur-genetec
Copy link
Copy Markdown
Member

@alafleur-genetec alafleur-genetec commented May 14, 2026

Summary

  • Rename the misspelled CopyConfiguratonSample folder and project to CopyConfigurationSample.
  • Update the solution project name and path for CopyConfigurationSample.
  • Align sample assembly names with their folder/project names:
    • DiagnosticServerSample now builds DiagnosticServerSample.exe.
    • FileCryptingManagerSample now builds FileCryptingManagerSample.exe.

These changes keep sample folder names, project names, and produced assembly names consistent.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR corrects a typo in the Platform SDK sample name by aligning the folder, project, and certificate filenames to CopyConfigurationSample, and updates the main samples solution to reference the corrected project path/name.

Changes:

  • Renamed the CopyConfiguratonSample sample to CopyConfigurationSample (folder/project/certificate naming alignment).
  • Updated Samples/Genetec.Dap.CodeSamples.sln to reference the corrected project name and path.
  • Ensured the sample’s entrypoint and supporting files are present under the corrected folder.

Reviewed changes

Copilot reviewed 1 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Samples/Platform SDK/CopyConfigurationSample/Program.cs Entry point for the renamed sample (top-level statements).
Samples/Platform SDK/CopyConfigurationSample/EntityManagerExtensions.cs Helper extension to await copy-configuration completion/progress events.
Samples/Platform SDK/CopyConfigurationSample/CopyConfigurationSample.csproj Project file under corrected name/path (targets, references, shared import, cert copy target).
Samples/Platform SDK/CopyConfigurationSample/CopyConfigurationSample.cs Sample implementation class aligned with corrected naming.
Samples/Platform SDK/CopyConfigurationSample/Certificates/CopyConfigurationSample.exe.cert Certificate file renamed to match corrected sample name.
Samples/Platform SDK/CopyConfigurationSample/App.config .NET Framework runtime configuration for the sample.
Samples/Genetec.Dap.CodeSamples.sln Solution updated to reference the corrected project name and path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alafleur-genetec alafleur-genetec force-pushed the fix/copy-configuration-typo branch from e4f5125 to 2823164 Compare May 16, 2026 04:32
@alafleur-genetec alafleur-genetec changed the title fix: correct CopyConfigurationSample folder and project name typo fix: align sample project and assembly names May 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants