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.

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

Formatter: fix to separate defs with visibility modifiers #4134

Merged
merged 2 commits into from Jun 4, 2017
Merged

Formatter: fix to separate defs with visibility modifiers #4134

merged 2 commits into from Jun 4, 2017

Conversation

makenowjust
Copy link
Contributor

Fixed #4133

@asterite asterite mentioned this pull request Mar 23, 2017
Closed
@makenowjust
Copy link
Contributor Author

Fixed #4133 (comment)

@Sija
Copy link
Contributor

Sija commented Apr 15, 2017

Is this PR GTG?

@asterite
Copy link
Member

asterite commented Jun 4, 2017

@makenowjust ありがとう！

@asterite asterite merged commit 430bf0b into crystal-lang:master Jun 4, 2017
@bcardiff bcardiff added this to the Next milestone Jun 5, 2017
@makenowjust makenowjust deleted the fix/crystal-format/modifier-def branch June 5, 2017 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
topic:tools:formatter
Projects
None yet
Milestone
0.23.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@makenowjust @Sija @asterite @bcardiff