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
feat: Export defineConfig defines the auxiliary function of the configuration #4127
Conversation
@lukastaegert I added some documentation at the end |
Thank you for your contribution! ❤️You can try out this pull request locally by installing Rollup via npm install rxliuli/rollup#feat_defineConfig or load it into the REPL: |
Codecov Report
@@ Coverage Diff @@
## master #4127 +/- ##
=======================================
Coverage 98.12% 98.12%
=======================================
Files 201 201
Lines 7077 7077
Branches 2071 2071
=======================================
Hits 6944 6944
Misses 64 64
Partials 69 69
Continue to review full report at Codecov.
|
docs/01-command-line-reference.md
Outdated
@@ -200,6 +200,33 @@ export default commandLineArgs => { | |||
} | |||
``` | |||
|
|||
Config Intellisense |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should make this a ####
sub-heading so that it looks right on the web site
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise looks great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added: 555cac3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 👍
I would like to ask, if yes, is it possible to support ts type configuration files? If I create a new PR to support this feature, will it be merged? |
There is already a PR for this which got stalled: #3835 |
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers: #4125
Description