- Issue created by @sonam_sharma
- 🇮🇳India sonam_sharma
Appy uneccessary_code.patch to remove unnecssary code
- Issue was unassigned.
- Status changed to Needs work
almost 2 years ago 8:33am 3 July 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The issue summary should always describe what should be fixed/changed. Neither the title nor screenshots are sufficient to describe what the issue is, even in the case a patch is provided.
In the case of a bug, the description should also list the steps necessary to reproduce the issue, starting from when the module/theme is installed. - Assigned to Priya_Degwekar
- 🇮🇹Italy apaderno Brescia, 🇮🇹
Still, the issue summary must make clear what unnecessary code means and what exactly is the code that should be removed.
Furthermore, basing on the provided patch, this is not even a bug report. Commented out lines are allowed, and they do not cause any bug. (Then, the content of a .info.yml should not be considered code. I know that making it seems there is a bug in code could make the maintainers quickly check this issue, but let us not lie on something that is not a bug.)
- Issue was unassigned.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
I am resetting the Assigned field, since there has not been work on this issue in the past 18 days.
- Status changed to Needs review
over 1 year ago 10:16am 23 July 2023