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

Doc improvements from LA Clojure Meetup discussion #539

Closed
seancorfield opened this issue Sep 12, 2024 · 0 comments
Closed

Doc improvements from LA Clojure Meetup discussion #539

seancorfield opened this issue Sep 12, 2024 · 0 comments
Assignees
Labels
documentation I need to write something up!

Comments

@seancorfield
Copy link
Owner

seancorfield commented Sep 12, 2024

  • maybe move legacy 1.x stuff to lower down the readme since no one needs to know that now?
  • most clauses expect a sequence -- say vector (even tho' it can be a list)
  • such as :set -- not immediately obvious there's a connection to update!
  • quoted symbolic form -- doesn't work with the ' in the web app
    • it's also not obvious in getting started why this feature exists
  • with a parameter of 0 (not clear that this is the result) -- move nil inline ahead?
  • inline with single argument -- no, supports multiple elements
  • % syntax should use keywords or quoted symbols
  • (sql/format {:select [:x [:y :d] [:%z.e] [:%z.f :g]]}) -- don't need [] on %z.e here
  • (param x) doesn't work (symbolic form), nor does :params with symbols -- both should -- see param special syntax and :params opts do not support symbols #543
  • examples of composite/over -- should have example of/links to other helpers' equivalents
  • dialects: mysql stropping isn't readable!
  • enable quoting globally via set-dialect! -- example or link!
@seancorfield seancorfield self-assigned this Sep 12, 2024
@seancorfield seancorfield added the documentation I need to write something up! label Sep 12, 2024
seancorfield added a commit that referenced this issue Sep 22, 2024
seancorfield added a commit that referenced this issue Sep 22, 2024
Signed-off-by: Sean Corfield <[email protected]>
seancorfield added a commit that referenced this issue Sep 22, 2024
seancorfield added a commit that referenced this issue Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation I need to write something up!
Projects
None yet
Development

No branches or pull requests

1 participant