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

Support specifying :default spec in custom indentation rules #347

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

camsaul
Copy link
Contributor

@camsaul camsaul commented Aug 20, 2024

Sometimes the default fuzzy rule for ^with- of [[:inner 0]] is inappropriate, for example if you had a function like

(defn with-timezone [t zone]
  ...)
 
;;; this is a regular function and should follow `:default` list indentation rules
(with-timezone (java.time.OffsetDateTime/now)
               "US/Pacific")

This PR adds support for explicitly specifying :default formatting for a specific symbol

@weavejester
Copy link
Owner

Thanks for the PR. This seems like a useful addition. Can you change the commit message to:

Add ':default' custom indentation rule

This puts it under 50 characters.

@camsaul camsaul force-pushed the support-default-spec branch from 47c1776 to 47715d7 Compare August 20, 2024 15:54
@camsaul
Copy link
Contributor Author

camsaul commented Aug 20, 2024

@weavejester done

@weavejester weavejester merged commit c832761 into weavejester:master Aug 21, 2024
1 check passed
@weavejester
Copy link
Owner

Merged. Thanks for the work (again!)

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.

2 participants