Skip to content

Conversation

@brongulus
Copy link

Added a fix for quick-help.org not being found when using straight. Fixes the issue #110

@brongulus brongulus changed the title Fixes #110 Fix for #110 broken nano-help Jan 4, 2022
@brongulus brongulus changed the title Fix for #110 broken nano-help Fix for broken nano-help Jan 4, 2022
@rougier
Copy link
Owner

rougier commented Jan 5, 2022

Thanks! Given the relative small size of the help, maybe it would be better to store it in a variable inside the source. What do you think?

@brongulus
Copy link
Author

That's very reasonable and I should've done that. Pushed a fix, hope it's fine now.

@rougier
Copy link
Owner

rougier commented Jan 6, 2022

Thanks. I don't see the fix, did you push it?

@brongulus
Copy link
Author

I removed the need for extra variable altogether, now it directly looks in the correct directory, without modifying the load-path at all. Hope that's acceptable!

@rougier
Copy link
Owner

rougier commented Jan 17, 2022

Sorry, I was saying to have the whole help string inside but maybe it's a bad idea in the end. Concerning your fix, I'm afraid it is a bit specific to the install method or is it the common place to search for files?

@brongulus
Copy link
Author

It's a common place for straight users but you raise a valid point about this not working out for people who use other methods, since it's not a valid fix, I'll close the PR for now and think about incorporating other methods of installation in the future if possible. Apologies for the inconvenience.

@brongulus brongulus closed this Jan 18, 2022
@rougier
Copy link
Owner

rougier commented Jan 31, 2022

Sorry for the late answer and no need to apology. The issue is still valid since the help doesn't work as expected depending on the installation. This needs to be fixed. I'll reopen the issue to remember it needs to be fixed.

@rougier rougier reopened this Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants