Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Updating documentation in example notebooks #56

Merged
merged 10 commits into from
Sep 25, 2024

Conversation

michaeldenes
Copy link
Member

@michaeldenes michaeldenes commented Sep 24, 2024

Updating all documentation in example notebooks that require data.

With #55 , this is to close #51

@michaeldenes michaeldenes marked this pull request as ready for review September 24, 2024 13:02
@michaeldenes
Copy link
Member Author

Making this 'ready for review' so that I can see the readthedocs build.

@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.66%. Comparing base (05a08ce) to head (50f48cf).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #56   +/-   ##
=======================================
  Coverage   66.66%   66.66%           
=======================================
  Files           8        8           
  Lines         717      717           
=======================================
  Hits          478      478           
  Misses        239      239           
Flag Coverage Δ
unit-tests 66.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/examples/example_Italy_coast.ipynb Outdated Show resolved Hide resolved
michaeldenes and others added 4 commits September 24, 2024 18:00
Co-authored-by: Erik van Sebille <e.vansebille@uu.nl>
Co-authored-by: Erik van Sebille <e.vansebille@uu.nl>
…HTML hyperlinking. Rather, we can force markdown hyperlinking.
@michaeldenes
Copy link
Member Author

@erikvansebille are there any other changes for the PR, or happy to merge?

@erikvansebille
Copy link
Member

All good; happy for you to merge!

@michaeldenes michaeldenes merged commit a692124 into main Sep 25, 2024
5 checks passed
@michaeldenes michaeldenes deleted the update_examples_documentation branch September 25, 2024 07:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

examples
3 participants