Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Conversation

jebschiefer
Copy link

Closes #382

Please let me know if there is a better approach to this (like using spawn in getPythonVersion or something else).

@jebschiefer jebschiefer changed the title List atom version in apm --version List Atom version in apm --version Oct 16, 2015
@evjan
Copy link

evjan commented Oct 17, 2015

A great starting point!

It fails the build and you added no test for the new functionality. When I run the tests locally on OS X with your change they fail for the same reason as the build. I'm not quite sure why, did you manage to get them running locally @jebschiefer?

@jebschiefer
Copy link
Author

@evjan Thanks for checking this out. Looks like I was using a different version of node than Travis CI. I will look into this more this evening or tomorrow. It may have to do with the path returned by config.getResourcePath. If that's the case then I'll try the spawn approach.

@damieng
Copy link
Contributor

damieng commented Sep 30, 2016

No activity on this PR which would require merge conflict resolution and specs.

@damieng damieng closed this Sep 30, 2016
virtuoushub pushed a commit to virtuoushub/apm that referenced this pull request Sep 13, 2017
virtuoushub pushed a commit to virtuoushub/apm that referenced this pull request Sep 17, 2017
virtuoushub pushed a commit to virtuoushub/apm that referenced this pull request Sep 27, 2017
virtuoushub pushed a commit to virtuoushub/apm that referenced this pull request Sep 29, 2017
Closes atom#382
See atom#453 and atom#473
Returning unknown instead of an empty object based on @50Wliu's suggestion. See atom#745
virtuoushub pushed a commit to virtuoushub/apm that referenced this pull request Sep 29, 2017
Closes atom#382
See atom#453 and atom#473
Returning unknown instead of an empty object based on @50Wliu's suggestion. See atom#745
virtuoushub pushed a commit to virtuoushub/apm that referenced this pull request Sep 29, 2017
Closes atom#382
See atom#453 and atom#473
Returning unknown instead of an empty object based on @50Wliu's suggestion. See atom#745
virtuoushub pushed a commit to virtuoushub/apm that referenced this pull request Oct 17, 2017
Closes atom#382
See atom#453 and atom#473
Returning unknown instead of an empty object based on @50Wliu's suggestion. See atom#745
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.

4 participants