Skip to content
This repository was archived by the owner on Jan 20, 2019. It is now read-only.

New ember debug command#2

Closed
sivakumar-kailasam wants to merge 3 commits intoember-cli:masterfrom
sivakumar-kailasam:master
Closed

New ember debug command#2
sivakumar-kailasam wants to merge 3 commits intoember-cli:masterfrom
sivakumar-kailasam:master

Conversation

@sivakumar-kailasam
Copy link
Member

@bcardarella
Copy link
Contributor

@sivakumar-kailasam I suspect some of the analytics stuff that @twokul wrote could be purposed for this.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what format should the output be?
Ideally we optimize for readability as well.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should also detect and log.

  • watchman
  • path/to/node
  • path/to/npm
  • node_modules that exist above the current cwd()

@stefanpenner
Copy link
Contributor

👍

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

non i can think of.

@sivakumar-kailasam
Copy link
Member Author

@stefanpenner I was thinking of markdown since it would be easy to drag drop and good for readability in the issues. Here's a sample I generated from my ember-cli repo,
https://gist.github.com/sivakumar-kailasam/44b31ea3cf3bde3813d2

If this output looks good I'll raise a PR

@sivakumar-kailasam
Copy link
Member Author

@bcardarella The analytics module being a wrapper over insight might help in looking at stats over the user base anonymously and is currently limited for viewing to folks from the core team. May be we can start writing these analytics' tracking messages to a file so that whenever ember debug is run it could include those messages as well. This PR aims at providing a file in an issue so that the details are available to anyone viewing the issue.

@stefanpenner thoughts on this?

@twokul
Copy link
Contributor

twokul commented Jan 4, 2015

hm, I think build time of the trees + errors should be a part of ember-debug file. Analytic in Ember CLI s is just for gathering stats, it won't help you much to debug.

@stefanpenner
Copy link
Contributor

superseeded by ember doctor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants