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

Exclude reverts from suggestion #32

Open
revi opened this issue May 20, 2019 · 3 comments
Open

Exclude reverts from suggestion #32

revi opened this issue May 20, 2019 · 3 comments

Comments

@revi
Copy link

revi commented May 20, 2019

Countervandalism works does not necessarily reflect the user's area of interest — it's probably better not to include them in suggestions.

@revi
Copy link
Author

revi commented May 20, 2019

Edit summary (one example) has 'Reverted' in the summary and Rollback as a tag. Both could be used as a rollback detection.

@nettrom
Copy link
Owner

nettrom commented Jun 12, 2019

Thanks for bringing this up @revi! The bot does try to remove reverts from a user's edit history before attempting to use it to create suggestions, but there's room for improvement. One example is that we do not look very far back in a user's edit history, and if it's all reverts then we use them instead of looking further. I think that might've been what caused this in your case. Sorry about that!

I'm keeping this open to remind me that this is something to work on.

@revi
Copy link
Author

revi commented Sep 10, 2019

Then, how many 'fresh' contribs do I need to make before I get rid of the polluted suggestions?

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

No branches or pull requests

2 participants