-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Update guides/source/plugins.md [ci skip] #30261
Conversation
r? @pixeltrix (@rails-bot has picked a reviewer for you, use r? to override) |
guides/source/plugins.md
Outdated
@@ -458,13 +449,13 @@ Gem plugins currently in development can easily be shared from any Git repositor | |||
commit the code to a Git repository (like GitHub) and add a line to the Gemfile of the application in question: | |||
|
|||
```ruby | |||
gem 'yaffle', git: 'git://github.com/yaffle_watcher/yaffle.git' | |||
gem "yaffle", git: "git://github.com/rails/yaffle.git" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bogdanvlviv Could you change to https
protocol from git protocol? (And I'll close a request #30262) Thanks for your great work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yhirano55 Thanks
f895641
to
87666e5
Compare
@@ -458,13 +449,13 @@ Gem plugins currently in development can easily be shared from any Git repositor | |||
commit the code to a Git repository (like GitHub) and add a line to the Gemfile of the application in question: | |||
|
|||
```ruby | |||
gem 'yaffle', git: 'git://github.com/yaffle_watcher/yaffle.git' | |||
gem "yaffle", git: "https://github.com/rails/yaffle.git" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we keep the link outside the rails
org?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've changed it guided by that one day there might be a user by https://github.com/yaffle_watcher
No description provided.