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

List modules #79

Merged
merged 1 commit into from
Nov 19, 2015
Merged

List modules #79

merged 1 commit into from
Nov 19, 2015

Conversation

kritzcreek
Copy link
Owner

No description provided.

@kritzcreek
Copy link
Owner Author

@nwolverson @epost This basically lists all the directories in the projects output/ folder which corresponds to the modules that could potentially be imported.

Should help with nwolverson/atom-ide-purescript#24 in a "pragmatic" manner.

I also changed the format for the "list loaded modules" command so that it now returns a list of strings instead of a comma separated string. If there is anything else you'd like me to add to this one let me know.

@nwolverson
Copy link
Contributor

For nwolverson/atom-ide-purescript#24 I think this is basically being done already, seems to make sense to do it here. Not sure why that issue is even still open, must check.

@epost
Copy link
Contributor

epost commented Nov 16, 2015

Looks neat. 😺

kritzcreek added a commit that referenced this pull request Nov 19, 2015
@kritzcreek kritzcreek merged commit 6de92d0 into master Nov 19, 2015
@kritzcreek kritzcreek deleted the list-modules branch November 19, 2015 02:32
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.

3 participants