clojure-lsp/clojure-lsp

Rename protocol parameters

Open

#1.692 aberto em 22 de out. de 2023

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Clojure (177 forks)batch import
enhancementgood first issue

Métricas do repositório

Stars
 (1.310 stars)
Métricas de merge de PR
 (Mesclagem média 47d 8h) (24 fundiu PRs em 30d)

Description

Per ericdallo's directions

Although inconsequential, it's sometimes useful to have all the implementation parameter names match the definition's.

Describe the solution you'd like When we rename a protocol's parameter declaration, all the known implementation's parameters would be renamed accordingly.

Describe alternatives you've considered Search-and-replace would certainly work, but protocols parameters frequently have very abstract names (this, coll, el) that are frequently found in various unrelated places.

Additional context I originally attempted to use LSP to rename a protocol's t to this in a large codebase because search-and-replace was matching various other ts (standing for timestamp)

Guia do colaborador