qdk icon indicating copy to clipboard operation
qdk copied to clipboard

Remove `w/=` and `w/` operations in our repo's Q# code

Open ScottCarda-MS opened this issue 7 months ago • 7 comments

Updates the Katas, Libraries, and Samples to remove w/= and w/. Copilot instruction file is also updated to instruct the copilot to avoid using these operators.

ScottCarda-MS avatar May 16 '25 21:05 ScottCarda-MS

Change in memory usage detected by benchmark.

Memory Report for 099fcb269df47ebe2d27a8c8d1f58879de1741d9

Test This Branch On Main Difference
compile core + standard lib 25228490 bytes 25218942 bytes 9548 bytes

github-actions[bot] avatar May 16 '25 22:05 github-actions[bot]

Of note with this one, we have a bit of a timing challenge: we should merge after the release with the feature goes out since the samples are treated as a live resource BUT we must merge before the release with the feature goes out so that the embedded samples have the new syntax. I think this means we have to hit that window of merging just before we know we are going to do an official release.

swernli avatar May 16 '25 22:05 swernli

Change in memory usage detected by benchmark.

Memory Report for 0f3ab13755297a1d817904b8808bcacd5591ee90

Test This Branch On Main Difference
compile core + standard lib 25228490 bytes 25218942 bytes 9548 bytes

github-actions[bot] avatar May 16 '25 23:05 github-actions[bot]

Change in memory usage detected by benchmark.

Memory Report for 78198eead84845fd43cc100287e6540beca7b143

Test This Branch On Main Difference
compile core + standard lib 25228490 bytes 25218942 bytes 9548 bytes

github-actions[bot] avatar May 16 '25 23:05 github-actions[bot]

@swernli why, what do you have in mind when you say "live resource"? Do we have direct links to the main of this repo from documentation?

minestarks avatar May 16 '25 23:05 minestarks

@swernli why, what do you have in mind when you say "live resource"? Do we have direct links to the main of this repo from documentation?

Yeah, last I checked we do. We still get referrals from learn.microsoft.com and quantum.microsoft.com, and I believe the learn site still deep links to the samples folder. It's possible those links have changed since I last looked though.

swernli avatar May 16 '25 23:05 swernli

We need to also consider that QCOM takes new packages and updates katas and samples regularly from them, but doesn't always update the engine/compiler when they take a new drop, so we'll need to coordinate with them to ensure we dont check in unless they'll be updating content and the compiler.

billti avatar May 19 '25 16:05 billti

Closing for now, keeping the branch and will reopen when/if this is rebased and ready to move forward.

swernli avatar Jun 23 '25 17:06 swernli