assaf / vanity

Experiment Driven Development for Ruby
http://vanity.labnotes.org
MIT License
1.55k stars 269 forks source link

Update template rendering and update_attributes for Rails 6.1; adds Appraisals for Rails 6.0 and 6.1 #366

Closed bensheldon closed 2 years ago

bensheldon commented 3 years ago
frostmark commented 2 years ago

Hey @phillbaker. Maybe I will fix the specs and we will merge that?

phillbaker commented 2 years ago

👋 I'm not actively working on this open source project any more.

@assaf I'm not sure if you've given other folks contributor access to this repo, if not, let's maybe open an issue and ask for volunteers?

frostmark commented 2 years ago

That would be nice!

bensheldon commented 2 years ago

I have repository access, but not to release on Rubygems. I can pick this up again if no one else is quicker to overcome the challenges.

bensheldon commented 2 years ago

@frostmark that would be fantastic if you can get this passing in local development.

I opened up #369 to try to get CI running again on the project.

phillbaker commented 2 years ago

I do have access to add you on ruby gems, can your confirm your username there?

On Tue, Jan 25, 2022 at 11:58 AM bensheldon @.***> wrote:

@frostmark https://github.com/frostmark that would be fantastic if you can get this passing in local development.

I opened up #369 https://github.com/assaf/vanity/pull/369 to try to get CI running again on the project.

— Reply to this email directly, view it on GitHub https://github.com/assaf/vanity/pull/366#issuecomment-1021403274, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAXCKN7FUAKR7ZFFMIBZSLUX3JBXANCNFSM4U2DEJIQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

bensheldon commented 2 years ago

Oops, I shouldn't have rebased this branch because I realized people have the old commit pinned 🤦 I just undid that so the original commit is in the branch.

bensheldon commented 2 years ago

Tests are green; I tried it on my own application. I guess it's ready 🚀

frostmark commented 2 years ago

That awesome!

bensheldon commented 2 years ago

I released this on rubygems in v3.0.1 too 🥳