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

Add web_console gemspec #77

Merged
merged 1 commit into from
Nov 25, 2014

Conversation

gsamokovarov
Copy link
Collaborator

Back in the days I haven't read the name your gem guide. We have a
chance of fixing the name before release.

@gsamokovarov
Copy link
Collaborator Author

@chancancode Was that your idea?

@gsamokovarov gsamokovarov force-pushed the web_console_gemspec branch 7 times, most recently from 6f5fff6 to d6f2673 Compare November 25, 2014 21:49
# We need those for the testing application to run.
s.add_development_dependency "actionmailer", rails_version
s.add_development_dependency "activerecord", rails_version
For now all the functionallity is delegated, but we'll stop maintaining web-console in 3.0.
Copy link
Member

Choose a reason for hiding this comment

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

Idea from bcrypt-ruby/bcrypt-ruby#86


  #######################################################


  The `web-console` gem has been renamed to `web_console`.

  Instead of installing `web-console`, you should install

  `web_console`.  Please update your Gemfile and other

  dependencies accordingly as the legacy `web-console`

  gem will not be updated after version 3.0.


  #######################################################

@gsamokovarov
Copy link
Collaborator Author

@chancancode @matthewd I relaxed the binding_of_caller version and added the post_install_message deprecation notice. Can you take a final look 😄

@@ -8,20 +8,21 @@ Gem::Specification.new do |s|
s.authors = ["Charlie Somerville", "Genadi Samokovarov", "Guillermo Iguaran", "Ryan Dao"]
s.email = ["[email protected]", "[email protected]", "[email protected]"]
s.homepage = "https://github.com/rails/web-console"
Copy link
Member

Choose a reason for hiding this comment

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

If you are set on the name I can rename the repo for you so you can point to https://github.com/rails/web_console

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

👍

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

🙇

Back in the days I haven't read the [name your gem] guide. We have a
chance of fixing the name before release.

[name your gem]: http://guides.rubygems.org/name-your-gem/
@gsamokovarov
Copy link
Collaborator Author

Any objections to merging this one?

gsamokovarov added a commit that referenced this pull request Nov 25, 2014
@gsamokovarov gsamokovarov merged commit 6e7af8b into rails:master Nov 25, 2014
@gsamokovarov gsamokovarov deleted the web_console_gemspec branch November 26, 2014 00:21
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.

2 participants