Closed jan-janssen closed 2 weeks ago
The pull request modifies the QueueAdapter
class in pysqa/queueadapter.py
by adding an execute_command
parameter to its constructor. This parameter is passed to the QueueAdapterCore
during initialization, enhancing its configurability based on the provided queue_type
. While the constructor logic is updated to accommodate this new parameter, the overall method signatures and functionality of the class remain unchanged. Error handling for configuration files continues to raise a ValueError
if required files are missing.
File | Change Summary |
---|---|
pysqa/queueadapter.py | Added execute_command parameter to the QueueAdapter constructor, modifying its initialization of QueueAdapterCore . |
QueueAdapter
class in pysqa/queueadapter.py
, specifically enhancing the constructor to accept additional parameters, which is directly related to the changes made in the main PR regarding the execute_command
parameter.🐇 In the code we hop and play,
New commands come out to stay.
With a parameter added, oh what fun!
QueueAdapter shines, our work is done!
Hooray for changes, bright and neat,
In the rabbit's world, they can't be beat! 🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
New Features
QueueAdapter
by allowing the passing of anexecute_command
parameter, improving command execution capabilities.Bug Fixes
queue.yaml
orclusters.yaml
are not found.