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

Better handling of tool dicussions #228

Closed
wants to merge 6 commits into from
Closed

Better handling of tool dicussions #228

wants to merge 6 commits into from

Conversation

mre
Copy link
Member

@mre mre commented Feb 5, 2019

We recently added guidelines on how to deprecate tools in #224 after a longer discussion in #223.
Since then I have learned a few things the hard way:

  1. Discussions can quickly get out of hand. It is not easy (and sometimes impossible) to find a solution that pleases everyone.
  2. The role of a project maintainer is to keep collaboration constructive and make all contributors feel welcome.

In light of that, I propose the following changes for contributing:

  1. ⚠️ will be replaced with ℹ️. We don't want to spread fear, we want to inform.
  2. As far as possible, project maintainers should be involved in the discussion to come to a fair conclusion.
  3. Discussions around tools will be linked in the README.md.
  4. Please remember that we're all human and we all have the same goal of making Open Source a little better.

@impredicative
Copy link
Collaborator

impredicative commented Feb 5, 2019

I understand the need for a civil discussion, but I fear this is an overreaction. If a tool doesn't reasonably support the current version of a language, and there is no hope that it soon will, I would consider its use deprecated. Injecting the tool's author into the discussion is risking a highly biased opinion. I think project maintainers should never be highlighted or called to a discussion. Calling them to the table was the problem.

@mre
Copy link
Member Author

mre commented Feb 6, 2019

I don't see why asking the maintainer if their tool is still maintained is a problem.
An opinion is always biased but the goal is to reach consensus.
From that perspective, to only way to reach a common ground is to hear all sides.

@impredicative
Copy link
Collaborator

impredicative commented Feb 6, 2019

We saw what happened when opinions from all sides were requested in the prior issue. It didn't help things. This is not a court of law where justice must be served at all costs. I support getting additional less-biased opinions from expert users, however. If for example you asked Fred Hoyle for his opinion on his steady state theory, he would defend it to his last breath no matter what, and it wouldn't help things. We do want to hear diverse opinions, but I think the opinions of expert users of the language should be sufficient.

@chadbrewbaker
Copy link
Contributor

Perhaps tagging tools with language version limitations? Javascript isn't alone. Several great C++ linters don't support C++17.

@mre
Copy link
Member Author

mre commented Mar 10, 2019

Perhaps tagging tools with language version limitations? Javascript isn't alone. Several great C++ linters don't support C++17.

Good point. Maybe there is a better way by providing more objective information on the tools. Here's hope that #194 brings more clarity in the future.

@mre mre closed this Mar 10, 2019
@mre mre deleted the deprecations branch January 28, 2020 22:30
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