easybuilders / easybuild-docs

EasyBuild documentation (mkdocs)
https://docs.easybuild.io
GNU General Public License v2.0
3 stars 23 forks source link

review hooks page #186

Open boegel opened 1 year ago

boegel commented 1 year ago

https://docs.easybuild.io/hooks/

jkwmoore commented 1 year ago

The example given in https://docs.easybuild.io/hooks/#manipulating-easyconfig-parameters looks suspect as self.cfg manipulation in the parse_hook shouldn't work.

Seems worth a correction with self.cfg.update('configopt', {'property': value})


Other thoughts about this page below:

Mostly as I am very rusty on Python but may also help people fairly new to Python, might be worth having this page include:

Various additional possibly generic but real world often used hook examples might also be a good shout for the page:

boegel commented 1 year ago

I'll take a look at @jkwmoore's suggestions, some are not relevant anymore, but there's some great feedback in there.