mention_system icon indicating copy to clipboard operation
mention_system copied to clipboard

Rails 7 Support

Open BilalBudhani opened this issue 2 years ago • 1 comments

BilalBudhani avatar Aug 17 '22 08:08 BilalBudhani

Why moving all dev and test dependencies from main gemspec to the several gemfiles? seems redundant to declare all the dependencies in all files

pmviva avatar Aug 22 '22 15:08 pmviva

Hi, this is exciting work. Please let me know how to help out. This is the only thing stopping me from updating my platform to rails 7. (I currently use it in rails 5)

Thank you so much and I look forward to the release!

azeemh avatar Dec 29 '22 15:12 azeemh

Hi, thanks for the feedback on the project. Regarding support for rails 7, i wanted to mention that I am not actually developing in ruby since a few years, and I don't have much spare time to invest in supporting this and the other three companion frameworks.

You could create a new PR with support for rails 7 and I can look at it and merge it. Sine i do not have time to keep up with further releases I was planning sooner or later deprecating this framework or transfer the ownership of the project to another developer who interested in maintain it.

Thanks Pablo

pmviva avatar Dec 30 '22 02:12 pmviva

i updated my gemfile for now and this doesn't have any bundle conflicts anymore.

#mention system gem -- updated for rails 7+
#gem 'mention_system'
gem 'mention_system', git: 'https://github.com/BilalBudhani/mention_system/', branch: 'rails-7'

I haven't been able to fully test because I still have to resolve version update on the has_friendship gem as well in my project...

azeemh avatar Dec 31 '22 20:12 azeemh

Why moving all dev and test dependencies from main gemspec to the several gemfiles? seems redundant to declare all the dependencies in all files

Thanks for the feedback. I moved all the dependencies to gemspec. I have also added CI to check against important versions of Rails.

You could create a new PR with support for rails 7 and I can look at it and merge it. Sine i do not have time to keep up with further releases I was planning sooner or later deprecating this framework or transfer the ownership of the project to another developer who interested in maintain it.

@pmviva Since I'm using this gem in a couple of my projects, I'm interested in taking it over and keeping the development going. Let me know what you think.

BilalBudhani avatar Jan 03 '23 05:01 BilalBudhani

Sure, we can discuss you taking over the project and supporting it from now on if you have the time and will

On Tue, Jan 3, 2023, 2:17 AM Bilal Budhani @.***> wrote:

Why moving all dev and test dependencies from main gemspec to the several gemfiles? seems redundant to declare all the dependencies in all files

Thanks for the feedback. I moved all the dependencies to gemspec. I have also added CI to check against important versions of Rails.

You could create a new PR with support for rails 7 and I can look at it and merge it. Sine i do not have time to keep up with further releases I was planning sooner or later deprecating this framework or transfer the ownership of the project to another developer who interested in maintain it.

@pmviva https://github.com/pmviva Since I'm using this gem in a couple of my projects, I'm interested in taking it over and keeping the development going. Let me know what you think.

— Reply to this email directly, view it on GitHub https://github.com/pmviva/mention_system/pull/8#issuecomment-1369404115, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADZOCKPE4ARWREWEI2XOIRDWQOY5FANCNFSM56YRH6NA . You are receiving this because you were mentioned.Message ID: @.***>

pmviva avatar Jan 03 '23 05:01 pmviva

Sure thing. Meanwhile, Please merge this PR if everything looks good.

BilalBudhani avatar Jan 03 '23 07:01 BilalBudhani

@BilalBudhani The PR is already merged, I can add you as owner of the project in rubygems so you can release a new version of the gem, since I do not have my ruby environment setup as of now.

pmviva avatar Jan 03 '23 23:01 pmviva

@pmviva sounds good. My username on Rubygems.org is the same as Github.

BilalBudhani avatar Jan 04 '23 09:01 BilalBudhani

I have released 0.3.0 version on Rubygems.org

@pmviva Can you add my commit access to this repo so I can push release related changes to master?

BilalBudhani avatar Jan 10 '23 10:01 BilalBudhani