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

FIX: display note and section properly if a parent line exists #55

Open
wants to merge 2 commits into
base: 14.0
Choose a base branch
from

Conversation

damdam-s
Copy link

@damdam-s damdam-s commented May 12, 2023

thanks to #26, we can add notes and sections in options lines.
This PR allows the report (print) document to take care of this and put a margin-left to visually identify what's inside a configurable product
iScreen Shoter - Google Chrome - 230512101548

@damdam-s
Copy link
Author

@hparfr can you please activate the CI workflow there?

@codecov
Copy link

codecov bot commented May 12, 2023

Codecov Report

Merging #55 (8dacd6d) into 14.0 (b1bc0ef) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             14.0      #55   +/-   ##
=======================================
  Coverage   82.44%   82.44%           
=======================================
  Files          47       47           
  Lines         826      826           
  Branches      154      154           
=======================================
  Hits          681      681           
  Misses        127      127           
  Partials       18       18           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@damdam-s
Copy link
Author

@hparfr is the pre-commit error something to fix with a copier or something similar?

@damdam-s
Copy link
Author

@hparfr you have to approve the CI workflow each time I do a commit?

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.

1 participant