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

Enhancments in existing checks needed for more sophisticated setups and Bufixes (Replaces #88) #90

Merged
merged 4 commits into from
Nov 24, 2020

Conversation

Elias481
Copy link
Contributor

@Elias481 Elias481 commented Sep 24, 2020

Sadly I made a merge error and closed PR #88 while resetting my master ro upstream (and again even in this try the main error to work on master branch in fork...).

So here are the rebased changes.
For now I excluded the topic that overlaps some way with #89 , I have to copare both changes and see how far they overlap and how to implement the stuff that is most probably missing in #89 if it addreeses same thing or how to integrate my adjustments on top of #89.

@willemdh willemdh mentioned this pull request Sep 25, 2020
@willemdh willemdh merged commit 4eb71e5 into district09:master Nov 24, 2020
@willemdh
Copy link
Collaborator

@Elias481 Sry it took so long. Just tested this on our Netapp and it seems to work fine.

@willemdh
Copy link
Collaborator

@Elias481 If you give me your full name, I'm ok with adding your name to the contributors.

@Elias481
Copy link
Contributor Author

@willemdh fine, just have to care for checking the functionality thats now out because of pending verify and merge with #89
My name is Elias Ohm.

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.

2 participants