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

Leave ActiveRecord::Base out of diagrams? #59

Closed
ianpetzer opened this issue Feb 20, 2014 · 13 comments
Closed

Leave ActiveRecord::Base out of diagrams? #59

ianpetzer opened this issue Feb 20, 2014 · 13 comments

Comments

@ianpetzer
Copy link

Hi,

Really love the diagrams that railroady is producing. Great Work!

Was just wondering if its possible to exclude ActiveRecord::Base out of the diagrams as it is associated with every model in the models and cluttering it up a little.

Thanks

@luxflux
Copy link

luxflux commented Mar 15, 2014

Yep, interested in this too...

@ghost
Copy link

ghost commented Mar 21, 2014

The rake task uses the -i option by default.

vagrant@precise64:/vagrant$ bundle exec rake diagram:models:complete
Generating /vagrant/doc/models_complete.svg
railroady -ilamM | sed -r 's/\x1B\[([0-9]{1,2}(;[0-9]{1,2})?)?[m|K]//g' | dot -Tsvg > /vagrant/doc/models_complete.svg
vagrant@precise64:/vagrant$

just reuse the railroady command line above, removing the -i option...

@ronwsmith
Copy link

Thanks @pfspontus! Would love a way to do this via the rake task.

@ghost
Copy link

ghost commented Jun 13, 2014

@ronwsmith I put it in our fork a while back https://github.com/bredband2, it's really easy to do.

@ronwsmith
Copy link

Thanks again @pfspontus. Have you submitted a PR to see if @preston will merge it?

@sergiopantoja
Copy link

+1 Doing this via the rake task would be helpful for me as well. Removing the -i option as @pfspontus recommended is working in the meantime.

@sio4
Copy link

sio4 commented Jul 11, 2014

Thanks @pfspontus.

@preston
Copy link
Owner

preston commented Oct 11, 2014

Anyone have objections to making this the default behavior? I think it's a great idea!

@ghost
Copy link

ghost commented Oct 11, 2014

👍 on removing the -i option from the default behavior

@preston
Copy link
Owner

preston commented Oct 11, 2014

@sandipransing @anilo : Any objections to me making this change?

@preston
Copy link
Owner

preston commented Oct 14, 2014

Changed for v1.2.0, just released!

@preston preston closed this as completed Oct 14, 2014
@sio4
Copy link

sio4 commented Oct 15, 2014

Good news! thanks!

@ianpetzer
Copy link
Author

Oh brilliant. Good work!

On Wed, Oct 15, 2014 at 2:07 AM, Yonghwan SO notifications@github.com
wrote:

Good news! thanks!


Reply to this email directly or view it on GitHub
#59 (comment).

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

No branches or pull requests

6 participants