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

feature (workspace IDE config files) - new commands to write/delete config files in the workspace #7171

Merged
merged 27 commits into from
Mar 21, 2023

Conversation

GiladShoham
Copy link
Member

@GiladShoham GiladShoham commented Mar 19, 2023

Proposed Changes

This is still an EXPERIMENTAL feature!!!

This PR adds a new aspect (workspace-config-files) to write/delete config files to/from the workspae.
This is useful mostly for IDE support.
This aspect can write config files such as tsconfig.json, eslint config and prettier configs.
The aspect knows to calculate the correct config file for each component based on its env.
Then dedup the written config files to write as less as possible config files.
The aspect support registering new config writers by custom aspects in case you want to write more config files.

The aspect will write the real config files into node_modules/.cache directory then it will write extending config files in the workspace / component folders (files that pointing to the real config files).

At the moment this command might override your own config files in case you have such.
So please make sure to run with --dry-run before running the actual command.

An example output can be seen here:
https://app.warp.dev/block/rikUfU0Vw0eXvJmJG1aqjX

@GiladShoham GiladShoham changed the title ide config files feature (workspace IDE config files) - new commands to write/delete config files in the workspace Mar 19, 2023
@GiladShoham GiladShoham merged commit 60580ea into master Mar 21, 2023
@GiladShoham GiladShoham deleted the ide-config-files branch March 21, 2023 10:46
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.

2 participants