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

Ignore additional metadata? #513

Closed
seancorfield opened this issue Nov 14, 2023 · 0 comments
Closed

Ignore additional metadata? #513

seancorfield opened this issue Nov 14, 2023 · 0 comments
Assignees
Labels
needs analysis I need to think about this!

Comments

@seancorfield
Copy link
Owner

From Slack: https://clojurians.slack.com/archives/C66EM8D5H/p1699978560075789?thread_ts=1698528126.920039&cid=C66EM8D5H

Sam H
on this, my work colleague ran into the metadata issue when something was adding :end-line and :end-column with numbers as values. (file was also there but that had a string value)
They were using doomacs so maybe something on cider / refactor-nrepl side added the extra key-values (edited)

seancorfield
It isn't so much about the values being numbers as the metadata being turned into SQL. I can add those to the ignore list but any "unknown" metadata would be treated as something to format into the generated SQL.

@seancorfield seancorfield added the needs analysis I need to think about this! label Nov 14, 2023
@seancorfield seancorfield self-assigned this Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs analysis I need to think about this!
Projects
None yet
Development

No branches or pull requests

1 participant