Skip to content

Conversation

@Nuzair46
Copy link
Member

To be merged after #384

@Nuzair46 Nuzair46 requested a review from pkuczynski January 28, 2026 10:59
pkuczynski
pkuczynski previously approved these changes Jan 28, 2026
pkuczynski
pkuczynski previously approved these changes Jan 28, 2026
Copy link
Member

@pkuczynski pkuczynski left a comment

Choose a reason for hiding this comment

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

Rails when you create an app create also .github and .kamal folders?

@Nuzair46
Copy link
Member Author

Nuzair46 commented Jan 28, 2026

Rails when you create an app create also .github and .kamal folders?

Yes, these folders are created by default by rails new command. I kept them as is. Its also created for the previous versions.

@Nuzair46
Copy link
Member Author

@pkuczynski Please check, All tests are passing now and I used minimal rails app for 8.1

@Nuzair46 Nuzair46 enabled auto-merge (squash) January 28, 2026 11:27
Copy link
Member

@pkuczynski pkuczynski left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks

@Nuzair46 Nuzair46 merged commit 68f492b into master Jan 28, 2026
15 checks passed
@Nuzair46 Nuzair46 deleted the add-rails-8.1-support branch January 28, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants