Skip to content

Conversation

@heyvaleria
Copy link
Contributor

After this Slack conversation, we realised that Suspenders is not currently recommended after all.
https://thoughtbot.slack.com/archives/C05DN55CPL3/p1747661858395259

Copy link
Contributor

@stevepolitodesign stevepolitodesign left a comment

Choose a reason for hiding this comment

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

I'd love to get more feedback on this, as I think the only controversial take is we're now suggesting we prefer Minitest over RSpec for new applications. There's an open discussion about this.

@heyvaleria
Copy link
Contributor Author

Thanks for you input, @stevepolitodesign
That's why I wished you opened this PR 😉

I did want to get this going though, because I do not want to find myself or anyone else starting a new app for a client and running into all those issues by using suspenders, and getting a bunch or extra stuff we likely don't need in a fresh rails app.
It is extra time and extra struggle, and it doesn't look good with the client either.

@stevepolitodesign
Copy link
Contributor

I'll also add that we have not used Suspenders for several new Team Rocker projects, so we're already not following the guidance.

@heyvaleria
Copy link
Contributor Author

cc @samithoughtbot @slickepinne ☝️

Copy link
Contributor

@samithoughtbot samithoughtbot left a comment

Choose a reason for hiding this comment

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

@heyvaleria thank you for taking the time to clarify this! This will help others going forward!

heyvaleria and others added 4 commits May 26, 2025 09:53
Co-authored-by: Steve Polito <stevepolitodesign@users.noreply.github.com>
Co-authored-by: Steve Polito <stevepolitodesign@users.noreply.github.com>
@heyvaleria heyvaleria force-pushed the edit_recommendation_suspenders branch from fe24604 to 9daa37b Compare May 26, 2025 07:53
@heyvaleria heyvaleria merged commit 0dc501c into main May 29, 2025
2 checks passed
@heyvaleria heyvaleria deleted the edit_recommendation_suspenders branch May 29, 2025 07:53
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.

7 participants