Comments (3)
ericdallo
commented on June 29, 2024
I wonder if that's the desired behavior always but maybe most of the time, what if someone wants to rename only the local var of t of that protocol and not all the implementations?
maybe we could do that and if anyone complains add a setting for the old way (although there was never a old way for protocols as we don't support rename on function arities of protocols)
from clojure-lsp.
aisamu
commented on June 29, 2024
I wonder if that's the desired behavior always but maybe most of the time, what if someone wants to rename only the local var of t of that protocol and not all the implementations?
The protocol declaration never has a body, so there would be no benefit to invoking a rename action vs just renaming the symbol directly!
from clojure-lsp.
ericdallo
commented on June 29, 2024
Yeah, for protocols that sounds safe
from clojure-lsp.
Related Issues (20)
- the suppress code action doesn't seem to interact well with toplevel forms that are preceded by a comment HOT 1
- Windows 11 - Clojure Language Client client: couldn't create connection to server. Message: Internal error Code: -32603 HOT 4
- Infer project type during initialization without need of a empty dependency file (bb.edn) HOT 1
- Not getting autocomplete for Java interop HOT 1
- Errors while trying to rename parameters of defprotocol function declarations HOT 3
- Support passing configurations to clj-depend via CLI
- Performance: allow disable keyword analysis for better memory usage
- Namespace alias shadows local variables HOT 1
- Exclude keys that are already included in the function call from auto-completion suggestions HOT 2
- Renaming deftype scrambles signature
- Launch performance is too slow to use as a pre-commit hook HOT 1
- Only most recently started clojure-lsp server is colorizing keywords HOT 3
- Drag-param does not work with multi-arity functions HOT 2
- max-line-length not respected when sorting large refer forms
- Enhance move-to-let to add or augment `:let` binding when in macros that support it HOT 2
- clojure-lsp-native-linux-aarch64.zip (2023.10.30-16.25.41) doesn't contain the latest version HOT 1
- Unable to require suggested namespace when `ns` isn't present
- Support correct namespace for exported clj-kondo hooks
- Add "Create clj-kondo hook" actions HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from clojure-lsp.