Skip to content

DOC: Config arg no longer reuired - #1874

Merged
nourinmohd merged 1 commit into
equinor:mainfrom
nourinmohd:1872_doc_config_arg
Oct 5, 2026
Merged

nourinmohd merged 1 commit into
equinor:mainfrom
nourinmohd:1872_doc_config_arg

Conversation

@nourinmohd

@nourinmohd nourinmohd commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Resolves #1872

Updates documentation to reflect automatic configuration discovery:

  • config is now documented as deprecated.
  • Discovery checks the nearest .fmu/, then the standard global config.
  • Examples no longer load or pass global configuration explicitly.
  • Updated the examples to use the latest schema.

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 ✅

@codecov-commenter

codecov-commenter commented Oct 2, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.67%. Comparing base (19d2f14) to head (52cf0fa).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1874   +/-   ##
=======================================
  Coverage   97.67%   97.67%           
=======================================
  Files          59       59           
  Lines        3748     3748           
=======================================
  Hits         3661     3661           
  Misses         87       87           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nourinmohd
nourinmohd force-pushed the 1872_doc_config_arg branch 2 times, most recently from d943dae to 52cf0fa Compare October 2, 2026 10:08
@nourinmohd
nourinmohd requested review from GibranAlfa and tnatt October 2, 2026 10:08

@tnatt tnatt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🧹

@nourinmohd
nourinmohd force-pushed the 1872_doc_config_arg branch from 52cf0fa to ef79fc2 Compare October 5, 2026 06:16
@nourinmohd
nourinmohd merged commit 39636b8 into equinor:main Oct 5, 2026
14 checks passed
@nourinmohd
nourinmohd deleted the 1872_doc_config_arg branch October 5, 2026 06:20
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.

Update documentation: config argument is no longer required

4 participants