rustworkx icon indicating copy to clipboard operation
rustworkx copied to clipboard

Erroneous Reno in tests

Open prakharb10 opened this issue 3 years ago • 3 comments

Information

  • retworkx version: 0.11.0
  • Python version: 3.9.6
  • Rust version: 1.63.0
  • Operating system: macOS 12.5.1

What is the current behavior?

There's a reno placed in the test directories at - https://github.com/Qiskit/rustworkx/blob/main/tests/rustworkx_tests/generators/releasenotes/notes/d-1-heavy-hex-e2e44861dc75009a.yaml

I'm not sure, but I think this is erroneous.

What is the expected behavior?

Reno should be placed in the releasenotes directory.

prakharb10 avatar Sep 15 '22 14:09 prakharb10

This is a good catch. We should fix this, the complexity is that this was included in a release already which means if we simply git mv the file to releasenotes.notes reno wiill treat this as part of the next release instead of the previous one. I think to fix this we should copy the contents of the release note included in the previous release like: https://github.com/Qiskit/rustworkx/blob/main/releasenotes/notes/0.11/prepare-0.11-af688e532712c830.yaml and remove the stray file. Then when we build release notes the next time it will include the contents in the proper place.

mtreinish avatar Sep 19 '22 14:09 mtreinish

I think I understand @mtreinish. I'll try to make a PR with the minor fix. What about something similar to https://github.com/Qiskit/qiskit-terra/pull/8320 in the repo to prevent this in the future?

prakharb10 avatar Sep 19 '22 15:09 prakharb10

Yeah adding the validation job is a good idea, that will prevent this in the future. (and since I'm apparently the typical cause of these issues it'll be good to ensure more discipline on my part)

mtreinish avatar Sep 19 '22 18:09 mtreinish