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

"lodash style issue mangement" #200

Closed
GerHobbelt opened this issue Apr 8, 2020 · 3 comments
Closed

"lodash style issue mangement" #200

GerHobbelt opened this issue Apr 8, 2020 · 3 comments
Labels
📖documentation Improvements or additions to documentation 👀FYI only 🧑‍🤝‍🧑help wanted🧑‍🤝‍🧑 Extra attention is needed. 🕵investigate Needs further analysis to find the root cause. 🤔question Further information is requested or this is a support question 🕵TLC Needs some special attention 👮wontfix This will not be worked on
Milestone

Comments

@GerHobbelt
Copy link
Collaborator

https://twitter.com/samselikoff/status/991395669016436736

@GerHobbelt GerHobbelt added the 🤔question Further information is requested or this is a support question label Apr 8, 2020
@GerHobbelt GerHobbelt added this to the v82 milestone Apr 8, 2020
@GerHobbelt
Copy link
Collaborator Author

(reminder for self: in Twitter click "Show this thread". for the next time you're confused. 😉 )

@GerHobbelt
Copy link
Collaborator Author

(came by this via 11ty/eleventy#379 (comment))

@GerHobbelt GerHobbelt added duplicate This issue or pull request already exists ⛷performance Anything that's related to UX: speed of response; I/O speed, etc. 🐛bug Something isn't working 👀FYI only 👮invalid This doesn't seem right 👮wontfix This will not be worked on 📖documentation Improvements or additions to documentation 🕵code review When the issue popped up due to code review or when (larger) code review is required. 🕵investigate Needs further analysis to find the root cause. 🕵TLC Needs some special attention 🦸‍♀️enhancement🦸‍♂️ New feature or request 🧑‍🤝‍🧑help wanted🧑‍🤝‍🧑 Extra attention is needed. labels Aug 21, 2020
@GerHobbelt
Copy link
Collaborator Author

GerHobbelt commented Aug 21, 2020

Conclusion after having considered this and inspected the way I work with issues:

It's not a good approach (for me at least). Why?

  • closed means closed. As in: done.

    At least that is how I treat github issues in any project: closed status is one of the strongest signals the issue tracker has and I don't want to increase the potential chaos by watering that signal down to a level of possibly maybe not.

  • We've got labels for that sort of thing: I took the idea of Unicode emoji enhanced labels off another project (forgot which one) where it worked very well; don't recall if I ripped off their emoji choices too, but I've set this set of labels to cover the 'lodash' idea (and then a whole bunch more):

duplicate ⛷performance 🐛bug 👀FYI only 👮invalid 👮wontfix 📖documentation 🕵code review 🕵investigate 🕵TLC 🦸‍♀️enhancement🦸‍♂️ 🧑‍🤝‍🧑help wanted🧑‍🤝‍🧑
  • the label called

🦸‍♀️enhancement🦸‍♂️

is specifically useful for this: anyone can filter issues on label and thus produce the set of feature requests. By not degrading the power of the closed state signal you can also easily discover which features have been added to the software and which ones are still pending for whatever reason.

  • I think the unmentioned benefit of the lodash approach to issue management is a premium score on the 'issues resolved/closed' KPI which is used by many OSS monitoring and reporting sites on the liveliness / activity of an OSS project, artificially raising the quality metrics. Smell like a "perfide prikkel" (perfidious stimulus) to me.

So after having considered it and its mentioned and unmentioned merits, I decide not to go that way.

At the time of this writing, Qiqqa has 126 open issues, many of which would become 'closed' if I were to use the lodash process, but most of these address very tough upgrade paths which need quite a bit of work (and not just at level of keyboard-🐒 😉 ). While it doesn't make me happy, I still think it is better to keep them open, including their near-duplicates, until the issue has been fully addressed and thus can be closed for real.

Any follow-up (bugs in the new work) from such actions can then, as usual, be filed in new issues, which can reference the closed issues, but I'd rather measure progress that way than artificially 'disappearing' the issues here. (Yes, while I felt immediately attracted to the lodash approach, it also feels artificial at a deep ethical level, but that's just me and my inner workings 🤡 )

So... this has been considered and WILL NOT be done.

@GerHobbelt GerHobbelt removed duplicate This issue or pull request already exists ⛷performance Anything that's related to UX: speed of response; I/O speed, etc. 🐛bug Something isn't working 👮invalid This doesn't seem right 🕵code review When the issue popped up due to code review or when (larger) code review is required. 🦸‍♀️enhancement🦸‍♂️ New feature or request labels Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📖documentation Improvements or additions to documentation 👀FYI only 🧑‍🤝‍🧑help wanted🧑‍🤝‍🧑 Extra attention is needed. 🕵investigate Needs further analysis to find the root cause. 🤔question Further information is requested or this is a support question 🕵TLC Needs some special attention 👮wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant