Skip to content

Open link in new tab#358

Merged
dstansby merged 3 commits intomainfrom
paddyroddy-patch-1
Jun 7, 2024
Merged

Open link in new tab#358
dstansby merged 3 commits intomainfrom
paddyroddy-patch-1

Conversation

@paddyroddy
Copy link
Member

No description provided.

@paddyroddy paddyroddy added the enhancement New feature or request label Apr 25, 2024
@paddyroddy paddyroddy requested a review from dstansby April 25, 2024 12:48
@paddyroddy paddyroddy self-assigned this Apr 25, 2024
@paddyroddy
Copy link
Member Author

Error is to do with actions/setup-python#850

@dstansby
Copy link
Member

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

@paddyroddy
Copy link
Member Author

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

@samcunliffe
Copy link
Member

samcunliffe commented Apr 25, 2024

Doesn't work?

Screen.Recording.2024-04-25.at.15.16.25.mov
➤ git status
On branch paddyroddy-patch-1
Your branch is up to date with 'origin/paddyroddy-patch-1'.

@paddyroddy
Copy link
Member Author

Doesn't work?

Click on Source repository top right

@paddyroddy
Copy link
Member Author

paddyroddy commented Apr 25, 2024

Individual links have to be handled in a different manner

@paddyroddy paddyroddy requested a review from samcunliffe April 25, 2024 14:23
@samcunliffe
Copy link
Member

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.

@paddyroddy
Copy link
Member Author

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

@giordano
Copy link
Member

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).

@dstansby dstansby merged commit a299d9d into main Jun 7, 2024
@dstansby dstansby deleted the paddyroddy-patch-1 branch June 7, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants