Skip to content

ENH: Alert users to set up stratigraphy after fmu init#98

Merged
GibranAlfa merged 1 commit intoequinor:mainfrom
GibranAlfa:alert-user-to-do-stratigraphy
Mar 6, 2026
Merged

ENH: Alert users to set up stratigraphy after fmu init#98
GibranAlfa merged 1 commit intoequinor:mainfrom
GibranAlfa:alert-user-to-do-stratigraphy

Conversation

@GibranAlfa
Copy link
Copy Markdown
Collaborator

Resolves #84

Checklist

  • Tests added (if not, comment why)
  • Test coverage equal or up from main (run pytest with --cov=src/ --cov-report term-missing)
  • If not squash merging, every commit passes tests
  • Appropriate commit prefix and precise commit message used
  • All debug prints and unnecessary comments removed
  • Docstrings are correct and updated
  • Documentation is updated, if necessary
  • Latest main rebased/merged into branch
  • Added comments on this PR where appropriate to help reviewers
  • Moved issue status on project board
  • Checked the boxes in this checklist ✅

**kwargs: Additional arguments to past to console.print().
"""
_print_stderr(_WARNING, *content, reason=reason, suggestion=reason, **kwargs)
_print_stderr(_WARNING, *content, reason=reason, suggestion=suggestion, **kwargs)
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also fix warning to use suggestion

@GibranAlfa GibranAlfa requested a review from mferrera March 6, 2026 12:08
@GibranAlfa GibranAlfa merged commit e23838e into equinor:main Mar 6, 2026
5 checks passed
@GibranAlfa GibranAlfa deleted the alert-user-to-do-stratigraphy branch March 6, 2026 12:30
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.

fmu init: Alert user that they need to set up their stratigraphy

2 participants