-
Notifications
You must be signed in to change notification settings - Fork 29
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
WSTAT warning and default well status #77
Conversation
…an atribute called faults, so I consider it is better to ask for the atribute than to ask if it was instancieated
… understand why it doesnt apears in my master repo
I'll do the code styling later |
This looks good @tayloris 🎉 As discussed, a small minor change suggested, then this is ready for merge 🚀
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Some minor suggestions.
Closes #69.
I include a warning message to let the user know well status are set OPEN by default.
However the message appears at the beginning of the run and I don't like the way is presented, so feel free to comment on that. we can discuss it on the meeting tomorrow.