ploomber / soorgeon

Convert monolithic Jupyter notebooks 📙 into maintainable Ploomber pipelines. 📊
https://ploomber.io
Apache License 2.0
78 stars 20 forks source link

display warning when only one H2 head detected. #56

Closed Wxl19980214 closed 2 years ago

Wxl19980214 commented 2 years ago

Describe your changes

Added a warning message if only one H2 head detected

Issue ticket number and link

Closes #47

Checklist before requesting a review

edublancas commented 2 years ago

can you add a test case? you can create a sample notebook with one H2 heading and check that the message is displayed: https://docs.pytest.org/en/6.2.x/capture.html

idomic commented 2 years ago

@Wxl19980214 I see some linting ci issues, please fix.

Wxl19980214 commented 2 years ago

test added and lint fixed.

idomic commented 2 years ago

Nice job!