Skip to content

Get extra attibutes from the model for the digraph export #194

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

Closed
wants to merge 1 commit into from

Conversation

froderik
Copy link

We use this gem to generate a dot file that we pick up with ruby_graphviz to show in a browser. Now we wanted to make nodes clickable and found out that the fastest path was to get more information into the dot file. In our case the URL and target attributes but the proposed solution can be used for any attribute.

@mceachen
Copy link
Collaborator

mceachen commented Jan 25, 2016 via email

@froderik
Copy link
Author

Yes! I had a look at specs but it turned out I needed to install MySQL to run them. Then I found tests.sh and tried the appraisal thing with only sqlite and then I get The git source https://github.com/matthuhiggins/foreigner.git is not yet checked out and run bundle install which I obviously already did. Any idea about how to move forward?

@pmkhougaz
Copy link

Hi @froderik, I recently encountered this very same problem so I thought I'd share a work-around if this is still an issue for you or anyone else viewing this. What worked for me was:

brew install mysql
ln -sfv /usr/local/opt/mysql/*.plist ~/Library/LaunchAgents 
brew services mysql start
bundle install
bundle exec rake spec

I am still not able to get the test matrix to run by executing the shell script ./tests.sh and as weird as it sounds, the steps above don't work when executed inside a tmux pane. Anyhow, I hope this helps!

@froderik
Copy link
Author

Aha - may be that I ran inside tmux actually. Good catch.

I will probably not get the time to work anymore on this now anyway so the monkey patch I added to our project will stay for a while. (As a consultant I am switching customer as of monday!)

@mceachen
Copy link
Collaborator

mceachen commented Dec 9, 2016

Please re-open when all tests pass.

@mceachen mceachen closed this Dec 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants