Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@ScottCarda-MS
Copy link
Contributor

@ScottCarda-MS ScottCarda-MS commented Feb 25, 2021

This PR removes the logic that throws an error when multiple entry points are present in a compilation.

Major Changes:

  • Removed logic that throws error when multiple entry points are present.
  • Made associated error codes obsolete.
  • Added a public-facing WriteFile method to CompilationLoader that allows files to be written even when they are not related to a source file. This is used by the runtime PR to write the new generated C# Entry Point file.
  • Updated the SDK targets file to include all C# files in the compilation, in-line with this guide (https://github.com/microsoft/qsharp-compiler/blob/main/src/QuantumSdk/README.md#injected-c-code).

This is should be merged in tandem with this PR on the runtime PR: microsoft/qsharp-runtime#528

Copy link
Contributor

@swernli swernli left a comment

Choose a reason for hiding this comment

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

LGTM

@bettinaheim bettinaheim dismissed bamarsha’s stale review March 23, 2021 17:12

Requested changes have been addressed.

@ScottCarda-MS ScottCarda-MS merged commit 361a53f into main Mar 23, 2021
@ScottCarda-MS ScottCarda-MS deleted the sccarda/MultipleEntryPoints branch March 24, 2021 22:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants