python-tooling icon indicating copy to clipboard operation
python-tooling copied to clipboard

Open link in new tab

Open paddyroddy opened this issue 1 year ago • 9 comments

paddyroddy avatar Apr 25 '24 12:04 paddyroddy

Error is to do with https://github.com/actions/setup-python/issues/850

paddyroddy avatar Apr 25 '24 12:04 paddyroddy

We don't have any way of viewing rendered docs from a PR to check them before merging do we?

dstansby avatar Apr 25 '24 13:04 dstansby

We don't have any way of viewing rendered docs from a PR to check them before merging do we?

I don't believe so

paddyroddy avatar Apr 25 '24 13:04 paddyroddy

Doesn't work?

https://github.com/UCL-ARC/python-tooling/assets/1836192/38e623a5-7b23-4d2b-adb2-12ae4ff16ada

➤ git status
On branch paddyroddy-patch-1
Your branch is up to date with 'origin/paddyroddy-patch-1'.

samcunliffe avatar Apr 25 '24 14:04 samcunliffe

Doesn't work?

Click on Source repository top right

paddyroddy avatar Apr 25 '24 14:04 paddyroddy

Individual links have to be handled in a different manner

paddyroddy avatar Apr 25 '24 14:04 paddyroddy

I like the idea, but I'd go with consistency. Only having the source link open in a new tab is... 🤷 meh. All external links? I could see the benefit.

samcunliffe avatar Apr 25 '24 14:04 samcunliffe

I like the idea, but I'd go with consistency. Only having the source link open in a new tab is... 🤷 meh. All external links? I could see the benefit.

But that would require going through every single link I believe, i.e. a different PR

paddyroddy avatar Apr 25 '24 14:04 paddyroddy

We don't have any way of viewing rendered docs from a PR to check them before merging do we?

For what is worth, we do that in the excalibur benchmarking repository: https://github.com/ukri-excalibur/excalibur-tests/blob/b53e6600c827ae797183c37edc5f7b598439b08c/.github/workflows/docs.yml (this is using mkdocs rather than jekyll, but to give an idea of what one can do to accomplish that).

giordano avatar Apr 25 '24 14:04 giordano