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

Justin's and My Changes #12

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Conversation

scott-steadman
Copy link
Collaborator

Justin updated the gemspec and version.

I fixed the tests, and added Registry._last_updated_at

ninjudd and others added 22 commits August 13, 2013 17:46
  This will indicate what time the registry was last updated.
  The README had references to plugin installation from berk's
  repository.
  rake db:migrate wasn't working
  I changed the references to me since Berk
  is, probably, no longer maintaining this gem.
  Add vendor/bundle to .gitignore so we don't see it when
  we run git status.
  I want to load acts_as_versioned as a gem.
  since that will externalize the dependency somewhat.
  These are Geni specific and don't belong here.
  We need to make sure we're running the correct ruby version.
  These weren't working because they used an old require line
  There were useless references.
  I need to set the gem versions since I've locked
  the ruby version to 1.9.
  Testing works.  There are a lot of failures though.
  The tests pass.

  I had to get the acts_as_versioned gem directly from github.
  This was the version I removed in an earlier commit.
  I had to convert from rcov to simplecov.
  Entering an invalid date caused the registry to raise an error
  which prevented our application from coming up.

  I now try to parse the date in three different formats, and if that
  fails I just return the string.
  It looks like gems specified in Gemfile
  aren't included when you include the Gem
  in another project.

  So I copied the acts_as_versioned file from
  the gem into our lib directory.
  There's something wonky with our config/registry.yml
  and I need some more details.

  I decided that adding a :verbose flag would be
  easier than rescuing the error, updating the messge
  and re-raising.
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