Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Changes for AHK_PATH #32

Closed
2 tasks done
spyoungtech opened this issue Jan 23, 2019 · 1 comment
Closed
2 tasks done

Changes for AHK_PATH #32

spyoungtech opened this issue Jan 23, 2019 · 1 comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@spyoungtech
Copy link
Owner

spyoungtech commented Jan 23, 2019 •
edited
Loading

Per action items out of #31 -- It would probably be nice to see some improvements for how this environment variable is documented/used. I think I want to:

  • update docs to be more clear on how AHK_PATH is used
  • raise an error (or warning?) on mis-configuration of of the variable

Alternatives might include

  • rename the variable
  • have this functionality removed/deprecated
  • add functionality to support either specify a directory or executable file.
@spyoungtech spyoungtech mentioned this issue Jan 23, 2019
Closed
@spyoungtech spyoungtech added help wanted Extra attention is needed good first issue Good for newcomers labels Jan 23, 2019
@spyoungtech spyoungtech mentioned this issue Feb 21, 2020
Merged
@yemreak yemreak mentioned this issue Feb 22, 2020
Merged
@spyoungtech
Copy link
Owner Author

There's now better error handling around this. Closing for now as it seems to no longer be a common issue for users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@spyoungtech