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.

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

Documentation for require() should be updated #13579

Closed
Gnimuc opened this issue Oct 13, 2015 · 3 comments · Fixed by #17249
Closed

Documentation for require() should be updated #13579

Gnimuc opened this issue Oct 13, 2015 · 3 comments · Fixed by #17249
Labels
docs This change adds or pertains to documentation good first issue Indicates a good issue for first-time contributors to Julia

Comments

@Gnimuc
Copy link
Contributor

Gnimuc commented Oct 13, 2015

currently, the doc says:

When searching for files, require first looks in the current working directory, then looks for package code under Pkg.dir(), then tries paths in the global array LOAD_PATH.

Since this commit was merged, require() hasn't been looking in the current working directory at all.

@Sisyphuss
Copy link

And I think

This function is part of the implementation of using / import

is not accurate.

It should be "This function uses using/import as part of its implementation".

@tkelman tkelman added the docs This change adds or pertains to documentation label Oct 13, 2015
@stevengj
Copy link
Member

@Sisyphuss, no, using/import call require.

@stevengj
Copy link
Member

It should probably also mention #13542 (it is case-sensitive, even on case-insensitive case-preserving filesystems like Mac and Windows).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation good first issue Indicates a good issue for first-time contributors to Julia
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants