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

Add --check cmd line option #35

Merged
merged 1 commit into from
Jan 13, 2019
Merged

Add --check cmd line option #35

merged 1 commit into from
Jan 13, 2019

Conversation

f18m
Copy link
Contributor

@f18m f18m commented Jan 12, 2019 •
edited
Loading

Address issue #27

lovesegfault
lovesegfault approved these changes Jan 13, 2019
View reviewed changes
Copy link
Owner

@lovesegfault lovesegfault left a comment

Choose a reason for hiding this comment

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

LGTM

@lovesegfault lovesegfault merged commit 21a02fa into lovesegfault:master Jan 13, 2019
f18m added a commit to f18m/beautysh that referenced this pull request Jan 14, 2019
@f18m
Add more unit tests to reformatter
5ae86aa 
Fix indent/outdent error for strings containing brackets (lovesegfault#36)

Add --check cmd line option (lovesegfault#35)

Enable TravisCI integration for tests

More elaborate unit tests

Fix travisCI

Fix parser problem with escaped quote characters

Fix parser problem with escaped quote characters
lovesegfault pushed a commit that referenced this pull request Jan 14, 2019
@f18m @lovesegfault
Add more unit tests to reformatter (#37)
1192b59 
Fix indent/outdent error for strings containing brackets (#36)

Add --check cmd line option (#35)

Enable TravisCI integration for tests

More elaborate unit tests

Fix travisCI

Fix parser problem with escaped quote characters

Fix parser problem with escaped quote characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lovesegfault lovesegfault lovesegfault approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@f18m @lovesegfault