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

Add support for config defined bindings #1740

Merged
merged 8 commits into from
Dec 3, 2024

Conversation

mwestphal
Copy link
Contributor

@mwestphal mwestphal commented Nov 28, 2024

  • Add support for configuring bindings in config file
  • Support order and multicommand
  • Add tests
  • Add about interactions and bindings configuration

Later:

Add user defined documentation support: #1747
Add glob support in config files: #1746
Add command-script option: #1734
Reenable test with command script:#1749
Improve command doc: #1749
Change minimal screenshot bind: #1715

@mwestphal mwestphal force-pushed the interaction_config_files branch 2 times, most recently from 6d8d79c to c2e5cd5 Compare November 28, 2024 21:37
Copy link

codecov bot commented Nov 28, 2024

Codecov Report

Attention: Patch coverage is 98.91892% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.72%. Comparing base (7a8ddb8) to head (8ade4ca).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
application/F3DStarter.cxx 99.17% 1 Missing ⚠️
library/public/interactor.h 97.29% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1740      +/-   ##
==========================================
+ Coverage   95.70%   95.72%   +0.01%     
==========================================
  Files         118      118              
  Lines        9392     9468      +76     
==========================================
+ Hits         8989     9063      +74     
- Misses        403      405       +2     

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

Copy link

You are modifying libf3d public API! ⚠️Please update bindings accordingly⚠️!
You can find them in their respective directories: python, java, webassembly.

@mwestphal mwestphal changed the title [draft] Interaction config files Add supporing for config defined bindings Nov 30, 2024
@mwestphal mwestphal changed the title Add supporing for config defined bindings Add support for config defined bindings Nov 30, 2024
@mwestphal mwestphal requested a review from Meakk December 1, 2024 07:54
application/F3DStarter.cxx Outdated Show resolved Hide resolved
application/F3DConfigFileTools.cxx Outdated Show resolved Hide resolved
application/F3DStarter.cxx Outdated Show resolved Hide resolved
application/F3DStarter.cxx Outdated Show resolved Hide resolved
library/public/interactor.h Outdated Show resolved Hide resolved
@mwestphal mwestphal requested a review from Meakk December 3, 2024 08:05
application/F3DOptionsTools.cxx Outdated Show resolved Hide resolved
application/F3DStarter.cxx Show resolved Hide resolved
library/public/interactor.h Outdated Show resolved Hide resolved
Works

renaming

Add tests

fixups

fixup tests

fixup

better testing

coverage

Revert "renaming"

This reverts commit 2324230.

Support order and multi commands

Fixups

fixups

clfi

fixups

again

doc

space

codespell

fixups coverage

last fixusp

codespell

review

clfi

Headeronly

Revert "Headeronly"

This reverts commit c25a76a.
@mwestphal mwestphal force-pushed the interaction_config_files branch from c25a76a to 8bf087e Compare December 3, 2024 11:22
@mwestphal mwestphal requested a review from Meakk December 3, 2024 15:01
@mwestphal mwestphal merged commit e6119e9 into f3d-app:master Dec 3, 2024
40 checks passed
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.

3 participants