Skip to content

[pull] master from ruby:master#44

Merged
pull[bot] merged 1 commit intosysfce2:masterfrom
ruby:master
Aug 10, 2025
Merged

[pull] master from ruby:master#44
pull[bot] merged 1 commit intosysfce2:masterfrom
ruby:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Aug 10, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

It appears the current behavior of disallowing links to non-text files
is deliberate, as it was explicitly added in
3628e19. While the commit message
explains the change, it doesn't provide a justification for excluding
non-text TopLevels. The issue mentioned in the commit message is also
unrelated to the change.

It's just as useful to link to a non-text file as it is to link to a
text file, so I think it should be allowed. I was surprised when it
didn't work. I want to use this feature in tilt's documentation.

If there is a reason to disallow it by default, I think this limitation
should be documented. I also I think we should add an option to allow it
in that case.
@pull pull bot locked and limited conversation to collaborators Aug 10, 2025
@pull pull bot added the ⤵️ pull label Aug 10, 2025
@pull pull bot merged commit 6639fe1 into sysfce2:master Aug 10, 2025
4 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant