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

Fix indentation (using ora-elisp-style-guide) #1612

Merged
merged 2 commits into from Jun 8, 2018

Conversation

raxod502
Copy link
Contributor

@raxod502 raxod502 commented Jun 8, 2018

colir.el Outdated
(function-item colir-compose-overlay)
(function-item colir-compose-soft-light)))
(function-item colir-compose-overlay)
(function-item colir-compose-soft-light)))
Copy link
Collaborator

@basil-conto basil-conto Jun 8, 2018

Choose a reason for hiding this comment

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

If you really must reindent this, then please place this and other first function-items on their own line.

Copy link
Contributor Author

@raxod502 raxod502 Jun 8, 2018

Choose a reason for hiding this comment

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

Done, in all appropriate places.

@abo-abo abo-abo merged commit d5eef62 into abo-abo:master Jun 8, 2018
1 of 2 checks passed
@abo-abo
Copy link
Owner

@abo-abo abo-abo commented Jun 8, 2018

Thanks.

@raxod502 raxod502 deleted the feat/ora-indent branch Jun 8, 2018
basil-conto added a commit to basil-conto/swiper that referenced this issue Aug 6, 2018
CONTRIBUTING.org: Suggest common-lisp-indent-function setting
similar to that of lisp-indent-function.
counsel.el (counsel--push-xref-marker):
ivy.el (ivy--dirname-p): Reindent accordingly.

Re: abo-abo#1601, abo-abo#1602, abo-abo#1612
abo-abo pushed a commit that referenced this issue Aug 6, 2018
CONTRIBUTING.org: Suggest common-lisp-indent-function setting
similar to that of lisp-indent-function.
counsel.el (counsel--push-xref-marker):
ivy.el (ivy--dirname-p): Reindent accordingly.

Re: #1601, #1602, #1612
Fixes #1702
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.

None yet

3 participants