Skip to content
This repository was archived by the owner on Aug 1, 2018. It is now read-only.

Fix gemspec#65

Open
rstacruz wants to merge 5 commits into
ejholmes:masterfrom
rstacruz:patch-1
Open

Fix gemspec#65
rstacruz wants to merge 5 commits into
ejholmes:masterfrom
rstacruz:patch-1

Conversation

@rstacruz

@rstacruz rstacruz commented Dec 2, 2015

Copy link
Copy Markdown

The current gemspec fails to build:

ERROR:  While executing gem ... (Gem::InvalidSpecificationException)
    duplicate dependency on activeadmin (~> 0.4.3, development), (>= 0.4.0) use:
    add_runtime_dependency 'activeadmin', '~> 0.4.3', '>= 0.4.0'

This fixes that. This also allows you to use this gem via GitHub:

gem 'active_admin_editor', github: 'ejholmes/active_admin_editor'

@rstacruz

rstacruz commented Dec 2, 2015

Copy link
Copy Markdown
Author

Also, I'm offering help to clean up the gem and release it, just let me know.

@ruprict

ruprict commented Mar 14, 2016

Copy link
Copy Markdown

@rstacruz I would say, just do it. Release it from your fork if needed. Clearly, @ejholmes has moved on from this gem.

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.

2 participants