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

turn off -st option, to allow for "perltidy -b $filename" #162

Merged
merged 1 commit into from
Jun 12, 2019

Conversation

karenetheridge
Copy link
Contributor

Summary

Now it's possible for the user to run perltidy manually.

Motivation

Because perltidy is mandatory for getting patches into this repository :)

References

none.

@marcusramberg
Copy link

I believe this repo is using the standard mojo perltidyrc, so that should probably be opened as a discussion upstream.

Sent with GitHawk

@jhthorsen jhthorsen requested a review from a team June 12, 2019 19:37
@jhthorsen
Copy link
Owner

I don't know why "-st" is part of the standard .perltidyrc. Does anyone have an argument for why it's there?

@jhthorsen jhthorsen requested review from jhthorsen and removed request for a team June 12, 2019 19:39
@jhthorsen
Copy link
Owner

Ah. Sorry. I messed up. Going to merge this PR 👍

@jhthorsen jhthorsen merged commit 653925b into jhthorsen:master Jun 12, 2019
@karenetheridge karenetheridge deleted the ether/fix-perltidyrc branch June 22, 2019 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants