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

feat: wrote first draft of statistics logging #454

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

elijah-potter
Copy link
Collaborator

As the authors of Harper, we want to know which rules are being most utilized, so we can properly allocate our time to what is actually useful to our users. This PR sets up a local-only log that keeps track of which lints are being applied and when.

This is done by creating an append-only file in the data directory (which is OS dependent).

This is not analytics. No data is being sent anywhere by harper-ls. That said, if you would like to submit your stats logs, it would be a huge help and we would welcome it.

@grantlemons
Copy link
Collaborator

Could be prudent to include an analysis tool in harper-cli? Nothing too crazy, but just creating a graph from >= 1 log file. I can make an issue and push it to the backlog if so.

@hippietrail
Copy link
Contributor

You should also log which rules are enabled and disabled etc. I don't know what settings there are yet.

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.

3 participants