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

A gem hasn't been released #2

Closed
wants to merge 1 commit into from
Closed

Conversation

brendon
Copy link

@brendon brendon commented Jun 19, 2019

Sorry to use a PR for this but issues are disabled on this repo.

Would you be able to release an actual gem for this project? At the moment it's uninstallable unless one references the GitHub repo in the Gemfile :)

@dmitry
Copy link

dmitry commented Aug 19, 2019

Would be nice to move this to the capistrano-plugins as well.

@brendon
Copy link
Author

brendon commented Aug 19, 2019

Here was my solution in the end. It's simple enough not to require a plugin:

append :linked_files, "config/master.key"

namespace :deploy do
  namespace :check do
    before :linked_files, :set_master_key do
      on roles(:app), in: :sequence, wait: 10 do
        unless test("[ -f #{shared_path}/config/master.key ]")
          upload! 'config/master.key', "#{shared_path}/config/master.key"
        end
      end
    end
  end
end

@dmitry
Copy link

dmitry commented Aug 19, 2019

@brendon thank you for sharing your solution. Totally makes sense!

@brendon
Copy link
Author

brendon commented Aug 19, 2019

You're most welcome! It's probably something that could be added to the capistrano-rails gem. I'll ping them and see what they think.

@brendon
Copy link
Author

brendon commented Aug 19, 2019

capistrano/rails#231

@dmitry
Copy link

dmitry commented Aug 21, 2019

@brendon awesome, it's been merged. Could be closed here.
@virgoproz please note for the others that it was merged into the latest capistrano: capistrano/rails#232

@brendon
Copy link
Author

brendon commented Aug 21, 2019

Hi @dmitry, unfortunately it was only a bit of documentation that was merged. It would still be good to see this as a gem release :)

@brendon
Copy link
Author

brendon commented Aug 30, 2024

I'll close this for now since the manually documented way must suit most peoples needs.

@brendon brendon closed this Aug 30, 2024
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