redhat-developer / redhat-developer/vscode-redhat-telemetry

Official recommendation of registering redhat.telemetry.enabled is against vscode official practices

Open
#137 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
31
Forks
39
Avg merge
9h 15m
Merged PRs (30d)
1

Description

It appears that only one extension is allowed to contribute a specific configuration property, such redhat.telemetry.enabled.

If user has more than one extension that tries to do this, it will popup a runtime warning message like below that complains about it:

AFAIK, the only solution known is not to register it and eventually use separate unique settings. The funny aspect is that we already have at least 3 extensions registering the same property:

What should we do to address this?

References

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing how redhat.telemetry.enabled is registered and compare the referenced VS Code, YAML, and Ansible issues. The work is done when the extensions no longer trigger the duplicate configuration-property warning and the chosen approach is documented or reflected across the affected extensions.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript, vscode
Domain
developer-experience, tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.