Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

making sure scrapli cleanup doesnt break things! #600

Merged
merged 7 commits into from
Sep 1, 2021

Conversation

carlmontanari
Copy link
Contributor

howdy folks!

@hellt is already in the loop on some of this, but for anyone else interested -- I've been doing a bit of housekeeping on scrapligo things and made some breaking-ish changes; this draft (until I finalize things ive been working on) is pinned to a dev branch and contains the little fixes that will be necessary if ya'll want to update the scrapligo pin. I think I should be fairly close to being done changing/breaking things and I will update this branch w/ pin to master branch (or perhaps even a real release finally) once I get to that point.

Carl

@carlmontanari carlmontanari marked this pull request as ready for review August 31, 2021 20:43
@hellt
Copy link
Member

hellt commented Sep 1, 2021

super dooper @carlmontanari
thanks

@hellt hellt merged commit 947e5b9 into srl-labs:master Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants