search

napalm-automation-community / napalm-fsos-ssh

SSH-based driver for FSOS (related products: see fs.com)
Other
1 stars 3 forks source link

this file shouldn't be empty, so I added the missing part #6

Closed lkthomas closed 2 years ago

M0NsTeRRR commented 2 years ago

Linked to #5

M0NsTeRRR commented 2 years ago

@lkthomas have you seen my review ? waiting for your change to merge the PR Don't forget to run pre-commit (check readme) to fix black issue

lkthomas commented 2 years ago

done, please check

M0NsTeRRR commented 2 years ago

@lkthomas you need to add the fix i've writed in the review, uncomment the import you did in the last commit and run pre-commit to fix file formatting.

Or if you prefer you can give me the permission to edit your PR (you have to check a box in your PR to allow mantainer to edit it)

lkthomas commented 2 years ago

"Allow edits by maintainers" is always checked