OpenMS / pyopenms-docs

pyOpenMS readthedocs documentation, additional utilities, addons, scripts, and examples.
https://pyopenms.readthedocs.io
Other
42 stars 52 forks source link

Get changed files action in test_pr.yml wrong #272

Closed jpfeuffer closed 1 year ago

jpfeuffer commented 2 years ago

It should ignore rsts in subfolders like _templates !!

jpfeuffer commented 2 years ago

see https://github.com/OpenMS/pyopenms-docs/pull/271

where I had to ignore it because the syntax of the action is so bad that I had to give up

jpfeuffer commented 1 year ago

@poshul didn't you recently have a look on that action in the OpenMS repo?

poshul commented 1 year ago

@jpfeuffer I just updated the get changed file action and limited depth of traversal. Can you double check that this does what you want?

jpfeuffer commented 1 year ago

Looks correct. You could test it with a whitespace PR changing files in the normal source/docs folder and templates.