Skip to content
This repository has been archived by the owner on Oct 27, 2021. It is now read-only.

Refactoring: Introduce spec & more modulare ns #43

Merged
merged 13 commits into from
May 15, 2020
Merged

Refactoring: Introduce spec & more modulare ns #43

merged 13 commits into from
May 15, 2020

Conversation

jerger
Copy link
Collaborator

@jerger jerger commented May 13, 2020

work is ready for a first inspection & discussion. The project now compiles, repl works & spec is in place for the mastodon part.

There might be the chance, that the whole thing works, but I've to find & fix my auth-issue, so I can not validate.

Nevertheless, I'm curious for your early review & comments :-)

@yogthos
Copy link
Owner

yogthos commented May 13, 2020

Overall, seems reasonable to me. I'm not sure there's much value in making config keywords namespaces though. Spec allows handling unqualified keys, documented in entry maps section. This would also address the issue of making them play nice with Js interop.

@jerger
Copy link
Collaborator Author

jerger commented May 14, 2020

Switching back to unqualified keywords was a really good hint :-)
I found some more places where I missed the use of the namespaced version.
Maybe I should search for automated refactoring tools in my setup ...

@yogthos
Copy link
Owner

yogthos commented May 14, 2020

Nice, and yeah there are a few decent refactoring tools for Emacs, clj-refactor and refactor-nrepl are both pretty good from what I hear. I use Cursive myself, but it's commercial. Definitely easier when you can refactor things automatically.

And happy to merge stuff if you think the changes are ready, I saw posting is confirmed to work. :)

@jerger
Copy link
Collaborator Author

jerger commented May 14, 2020

Let me do some additional test (e.g. posting images).
And I want to be able to reproduce #issue24 or confirm it's closed now.

@jerger
Copy link
Collaborator Author

jerger commented May 15, 2020

Okay, the pull request is ready to be merged from my point of view.

I finished testing (and found two more bugs in my refactoring :-)

#issue24 I can no longer reproduce - I will close it.

BTW: I'm using vs-code & calva. Refactoring support will arrive soon also on my env :-)

@yogthos
Copy link
Owner

yogthos commented May 15, 2020

Awesome, everything looks good to me as well. And Calva is pretty decent overall, I use it myself for some projects. It also works with clj-kondo, I recommend taking a look at it if you haven't. It's quite good at finding things like mismatched arguments, unused variables, name mismatches, and so on.

@yogthos yogthos merged commit 3f1f80e into yogthos:master May 15, 2020
@raboof raboof mentioned this pull request Jun 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants