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.

Sign up for GitHub

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

Jump to bottom

ForbidTStruct cop's autocorrect should support new lines and adhere to max line length #203

Merged
merged 3 commits into from Mar 6, 2024
Merged

ForbidTStruct cop's autocorrect should support new lines and adhere to max line length #203

merged 3 commits into from Mar 6, 2024

Conversation

KaanOzkan
Copy link
Contributor

No description provided.

@Morriar @KaanOzkan
Strip spaces from type definitions
cdfa443 
Signed-off-by: Alexandre Terrasa <alexandre.terrasa@shopify.com>
@Morriar @KaanOzkan
Split line if it's longer than max line length
0ef3f07
@KaanOzkan KaanOzkan marked this pull request as ready for review March 5, 2024 20:16
@KaanOzkan KaanOzkan requested a review from a team as a code owner March 5, 2024 20:16
@KaanOzkan KaanOzkan requested a review from Morriar March 6, 2024 17:39
@KaanOzkan KaanOzkan merged commit bd50ee1 into main Mar 6, 2024
11 checks passed
@KaanOzkan KaanOzkan deleted the at-tmp branch March 6, 2024 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andyw8 andyw8 andyw8 approved these changes

@Morriar Morriar Morriar approved these changes

@vinistock vinistock Awaiting requested review from vinistock vinistock is a code owner automatically assigned from Shopify/sorbet

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@KaanOzkan @andyw8 @Morriar