Skip to content

make protocol mapping None by default#809

Merged
atravitz merged 3 commits into
mainfrom
feat/update_mapping_default
Jul 1, 2026
Merged

make protocol mapping None by default#809
atravitz merged 3 commits into
mainfrom
feat/update_mapping_default

Conversation

@atravitz

@atravitz atravitz commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Tips

  • Comment "pre-commit.ci autofix" to have pre-commit.ci atomically format your PR.
    Since this will create a commit, it is best to make this comment when you are finished with your work.

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: yes / no
If yes, please provide details here:

Checklist

  • Added a news entry
  • Filled in the AI-generated code disclosure.

Developers certificate of origin

@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.64%. Comparing base (b18420e) to head (408b5f6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #809   +/-   ##
=======================================
  Coverage   98.64%   98.64%           
=======================================
  Files          43       43           
  Lines        2807     2807           
=======================================
  Hits         2769     2769           
  Misses         38       38           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@IAlibay IAlibay left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🎉🎉🎉

Comment thread src/gufe/protocols/protocol.py
@atravitz atravitz enabled auto-merge (squash) July 1, 2026 18:15
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

No API break detected ✅

View workflow run

Griffe output
$ griffe check "gufe" -s src --no-inspection --no-color --verbose -a origin/main

@atravitz atravitz merged commit 3ccfcbd into main Jul 1, 2026
13 checks passed
@atravitz atravitz deleted the feat/update_mapping_default branch July 1, 2026 18:27
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