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

[OE] plugins should not extend foundational configs unless they need it #6305

Open
kavilla opened this issue Apr 2, 2024 · 0 comments
Open
Labels
enhancement New feature or request good first issue Good for newcomers operational excellence

Comments

@kavilla
Copy link
Member

kavilla commented Apr 2, 2024

Is your feature request related to a problem? Please describe.

When forked, a habit was formed to either completely replace the tsconfig, or eslint file that it has by default or extending for not much purpose.

Build failures are likely to occur and delays in releasing if the decision to do this was made with insight on what they were doing. It also could force them to install dependencies that they do not need and we managed for them to avoid security issues.

Example:
opensearch-project/anomaly-detection-dashboards-plugin#702

Describe the solution you'd like

Audit https://github.com/opensearch-project plugins for OpenSearch Dashboards and validate the usages (or lack of) of the repo.

If so, create an issue in their repo (potentially make the change). Track it apart of this issue.

Describe alternatives you've considered

n/a

Additional context

n/a

@kavilla kavilla added enhancement New feature or request operational excellence labels Apr 2, 2024
@kavilla kavilla added good first issue Good for newcomers and removed untriaged labels Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers operational excellence
Projects
None yet
Development

No branches or pull requests

1 participant