Argument fix for single_connection method

[CaptainFaisal]

Check the 4th parameter of single_connection method's definition, it's pbc_mode, so we need to pass args.pbc as 4th argument here.

Summary by Sourcery

Fix the argument order in the call to the single_connection method by passing args.pbc as the fourth argument instead of args.pixie_dust.

Bug Fixes:

• Fix the argument passed to the single_connection method by replacing the incorrect third argument with the correct fourth argument, args.pbc.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request fixes an argument mismatch in the single_connection method call within the oneshot.py file. The change ensures that the pbc_mode parameter is correctly passed as the fourth argument.

File-Level Changes

Change	Details	Files
Corrected argument order in single_connection method call	Added args.pbc as the fourth argument to the single_connection method call Aligned the method call with its definition, which expects pbc_mode as the fourth parameter	oneshot.py

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @CaptainFaisal - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}