dracula / visual-studio-code

🧛🏻‍♂️ Dark theme for Visual Studio Code
https://draculatheme.com/visual-studio-code
MIT License
776 stars 208 forks source link

🌱 v2.25.0 #253

Closed luxonauta closed 1 month ago

luxonauta commented 1 month ago

Note: This update brings the Dracula Theme closer to what we have in Dracula PRO and thus facilitates support for both; it also prepares us to introduce the Light variant for the Classic in the future.

dsifford commented 1 month ago

@luxonauta I do not agree with your aggressiveness here. Who approved this?

luxonauta commented 1 month ago

Hey 👋🏻 @dsifford, Zeno and I have been working on these updates for PRO for a while, and we wanted to port them to Classic as quickly as possible to bring the Dracula Classic theme closer to what we have in Dracula PRO.

dsifford commented 1 month ago

I do not agree here. I emailed zeno just now and I will say that if this is the practice that you are going to use moving forward then I'm forking this project and leaving. This is way too aggressive and is not in the spirit of OSS.

A theme with 7+ million installs requires a lot more thought when making changes like this. I found out about this after seeing 10+ negative review this morning in the marketplace.

luxonauta commented 1 month ago

Hey 👋🏻 @dsifford,

I understand your concern and would like to clarify a few points. 🧩

From our perspective, the repo seemed abandoned, with several issues and PRs dating back two years. Despite trying to email you and commenting on several PRs and Issues, we are still waiting to receive a response (which is not your obligation, and we understand and respect your time).

However, the last commit was on July 15, 2023. One year later, with several updates to the PRO, we decided in December last year to bring Classic closer to what we have in Dracula PRO to facilitate support for both and actively support them closely. 😅

I agree with you, and I'll exercise more caution and thoughtfulness when making changes. If you agree, I will also look for your review on changes like this one.

Our shared goal is to collaborate and improve the project constructively.

✅ Also, I'll make sure to open separate PRs with the changes you suggested here as comments.

dsifford commented 1 month ago

A few things here.

  1. You have never emailed me.
  2. A theme is not something that changes often. Issue count should not be treated as abandonment. Lots of those issues were single-user reports requesting jarring changes. I was waiting until those issue received traction before moving forward on them.
  3. Dracula Pro is a fork of this repository. I'm flattered. But I do not agree with just taking the changes you made to that fork and blasting it back here without code review. This is not how software works. This feels to me like a hostile takeover and I very much do not appreciate almost a decade of my work getting steamrolled in one shot.

I understand that you're enthusiastic. That's very nice to see. But this is not how OSS works. Your changes here I do not agree with and unless they are rolled back and made in a way that is reviewable, then I'm forking this project and leaving.

@zenorocha I would love to hear your thoughts here. Do you agree with how this change was performed?

luxonauta commented 1 month ago

I'm sorry if there was any miscommunication. We attempted multiple times to reach out and get feedback on the PRs and issues, but I understand there may have been some misunderstandings. 😥

Our intention has always been to collaborate and improve the project constructively. I appreciate your work and dedication over the years, and I am committed to respecting and working together for the benefit of the project.

Also, the changes were made to contribute positively, and there was no intention to pressure or disrespect you.

✅ I am available to discuss and review any necessary changes to ensure we are all aligned.

dsifford commented 1 month ago

We attempted multiple times to reach out and get feedback on the PRs and issues,

Please show me.

luxonauta commented 1 month ago

I think it will be more beneficial to iterate together from now on than to continue this discussion.

I am available to discuss and review any necessary changes to ensure we are all aligned. 😅

GoodbyeNJN commented 1 month ago

I think it will be more beneficial to iterate together from now on than to continue this discussion.

Since there has been quite a bit of negative feedback and the change does not follow the contribution process, the immediate priority should be to roll back the code and release a new version to avoid disrupting more users.

As for iteration and discussion, that is something that should be considered after this.

luxonauta commented 1 month ago

I reverted the recent changes and will open separate PRs for review. Thank you all for the discussion and feedback! 🧩