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

Error upgrading from v16.2.1 to v16.3.0: No configuration provided #7577

Closed
darrinmn9 opened this issue Mar 25, 2024 · 5 comments · Fixed by #7578
Closed

Error upgrading from v16.2.1 to v16.3.0: No configuration provided #7577

darrinmn9 opened this issue Mar 25, 2024 · 5 comments · Fixed by #7578
Labels
status: wip is being worked on by someone type: bug a problem with a feature or rule

Comments

@darrinmn9
Copy link

What minimal example or steps are needed to reproduce the bug?

upon upgrading from v16.2.1 -> v16.3.0 (no other code/dependencies were changed), i get this error when linting files with stylelint

yarn stylelint --fix --allow-empty-input '**/*.{vue,css,scss}'

Error: No configuration provided for /app/src/app.vue
    at configurationError (file:///app/node_modules/stylelint/lib/utils/configurationError.mjs:12:49)
    at getConfigForFile (file:///app/node_modules/stylelint/lib/getConfigForFile.mjs:67:4)
    at async isPathIgnored (file:///app/node_modules/stylelint/lib/isPathIgnored.mjs:23:17)
    at async lintSource (file:///app/node_modules/stylelint/lib/lintSource.mjs:43:20)
    at async file:///app/node_modules/stylelint/lib/standalone.mjs:214:27
    at async Promise.all (index 0)
    at async standalone (file:///app/node_modules/stylelint/lib/standalone.mjs:257:22)
error Command failed with exit code 78.

What minimal configuration is needed to reproduce the bug?

// stylelint.config.js
module.exports = {
  extends: [
    'stylelint-config-standard-scss',
    'stylelint-config-standard-vue/scss',
    'stylelint-config-recess-order',
    'stylelint-config-css-modules',
  ],
  rules: {},
}

How did you run Stylelint?

yarn stylelint --fix --allow-empty-input '**/*.{vue,css,scss}'

Which Stylelint-related dependencies are you using?

    "stylelint": "16.3.0",
    "stylelint-config-css-modules": "^4.1.0",
    "stylelint-config-recess-order": "^4.0.0",
    "stylelint-config-standard-scss": "^12.0.0",
    "stylelint-config-standard-vue": "^1.0.0",
    "stylelint-scss": "^6.0.0",

What did you expect to happen?

No error upon the minor upgrade from v16.2.1 -> v16.3.0 (no other dependencies were changed)

What actually happened?

yarn stylelint --fix --allow-empty-input '**/*.{vue,css,scss}'

Error: No configuration provided for /app/src/app.vue
    at configurationError (file:///app/node_modules/stylelint/lib/utils/configurationError.mjs:12:49)
    at getConfigForFile (file:///app/node_modules/stylelint/lib/getConfigForFile.mjs:67:4)
    at async isPathIgnored (file:///app/node_modules/stylelint/lib/isPathIgnored.mjs:23:17)
    at async lintSource (file:///app/node_modules/stylelint/lib/lintSource.mjs:43:20)
    at async file:///app/node_modules/stylelint/lib/standalone.mjs:214:27
    at async Promise.all (index 0)
    at async standalone (file:///app/node_modules/stylelint/lib/standalone.mjs:257:22)
error Command failed with exit code 78.

Do you have a proposal to fix the bug?

No response

@Mouvedia Mouvedia added the status: needs investigation triage needs further investigation label Mar 25, 2024
@Mouvedia
Copy link
Member

Mouvedia commented Mar 25, 2024

getConfigForFile calls augmentConfigFull which may call augmentConfigBasic which calls absolutizePaths which may call getModulePath which has changed in the last version.
That's what I would investigate first.

@JounQin do you think that's related?

@ybiquitous
Copy link
Member

Related to #7576

@ybiquitous ybiquitous added status: wip is being worked on by someone type: bug a problem with a feature or rule and removed status: needs investigation triage needs further investigation labels Mar 26, 2024
@ybiquitous
Copy link
Member

Thanks for the report. Here's a reproducible demo, which raises:

Cannot read properties of undefined (reading 'ignoreFiles')

@daniluk4000
Copy link

I can confirm this is fixed for us

@darrinmn9
Copy link
Author

great job on such a quick fix! Thank you all so much for everything you do to make stylelint awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: wip is being worked on by someone type: bug a problem with a feature or rule
Development

Successfully merging a pull request may close this issue.

4 participants