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

Fix slow performance due to excessive calls to find-namespace #234

Merged

Conversation

LuisThiamNye
Copy link
Contributor

Ensure the find-namespace function is only called once per reformat request. Otherwise, it is called many times per line, significantly slowing down the formatter.

Addresses #226

@atomist
Copy link

atomist bot commented May 24, 2021

Commit messages of this repository should follow the seven rules of a great Git commit message, as mentioned in the project's contributing guidelines. It looks like there's a few issues with the commit messages in this pull request:

  • The commit message should not contain markdown.

@LuisThiamNye LuisThiamNye force-pushed the luistn/fix-perf-find-namespace branch from d199bc9 to 95ceb97 Compare May 24, 2021 17:42
Copy link
Owner

@weavejester weavejester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch, and apologies about the delay getting to it. Just have a couple of minor formatting issues and then I can merge it in.

cljfmt/src/cljfmt/core.cljc Outdated Show resolved Hide resolved
cljfmt/src/cljfmt/core.cljc Outdated Show resolved Hide resolved
Ensure this function is only called once per reformat request.
Otherwise, it is called many times per line, significantly slowing down
the formatter.
@LuisThiamNye LuisThiamNye force-pushed the luistn/fix-perf-find-namespace branch from 95ceb97 to 81bb59b Compare June 11, 2021 16:49
@LuisThiamNye LuisThiamNye requested a review from weavejester June 11, 2021 17:53
@weavejester weavejester merged commit 6b91d7b into weavejester:master Jun 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants