inbo / tutorials

A collection of technical tutorials for INBO (and anyone who's interested)
https://inbo.github.io/tutorials/
Creative Commons Attribution 4.0 International
22 stars 9 forks source link

add package dhcurve to inbo_software (issue #183) #293

Closed ElsLommelen closed 2 years ago

ElsLommelen commented 2 years ago

Description

I added a link and a description of package dhcurve to the webpage of INBO software. Package is in collaboration with @ThierryO and @leymanan

Related Issue

Part of #183

Task list

Previewing the pull request

Thanks to GitHub Actions, an artifact (=zip file) of the rendered website is automatically created for each pull request.

Instructions

1) On the PR page, you can find a "details" link under "checks - On PR, build the site and ...". Go there, click on the top link in the left sidebar ("Summary"), and download the generated artifact at the bottom of the page. 2) Decompress it and make sure the target directory is called 'tutorials' (you may need to rename it) 3) From the parent directory (just above the tutorials folder you created/renamed), run python -m http.server 8887, or launch the Google Chrome Web Server app and point it at the parent directory. 4) Point your browser to http://localhost:8887/tutorials. 5) Review the updated website and accept/refuse/comment the PR

Note: for step 3, you can use any other simple HTTP server to serve the current directory if you don't have a Python 3 environment or Google Chrome available.

ThierryO commented 2 years ago

Maybe at the package to https://github.com/inbo/universe too.

florisvdh commented 2 years ago

@ElsLommelen For now (perhaps awaiting #261 before merging): can you merge master in add-dhcurve (git checkout add-dhcurve && git merge --no-ff master) and push the new commit? That could solve the failed check; the add-dhcurve branch is relatively old.

ElsLommelen commented 2 years ago

@florisvdh oups, I did this locally before making this PR, but I forgot to push the commits... :-(

florisvdh commented 2 years ago

Excellent, thanks.

peterdesmet commented 2 years ago

This can be merged. I'll resolve conflicts in #261.