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

Features Override Warning #2705

Closed
mikemadison13 opened this issue Apr 2, 2018 · 1 comment
Closed

Features Override Warning #2705

mikemadison13 opened this issue Apr 2, 2018 · 1 comment
Labels
Bug Something isn't working

Comments

@mikemadison13
Copy link
Contributor

My system information:

  • Operating system type: [replace]
  • Operating system version: [replace]
  • BLT version: 9.x-dev f6c7d88

When I run this command:

blt setup:config-import

I get the following output:

[Acquia\Blt\Robo\Tasks\DrushTask] Done in 59.278s
[warning] Features override checks are currently disabled due to a Drush 9 incompatibility.

And I expected this to happen:
Considering I am using config-split now and not features, I am surprised that we are still checking for features overrides. This shouldn't be happening in this scenario.

@mikemadison13
Copy link
Contributor Author

See https://github.com/acquia/blt/blob/9.x/src/Robo/Commands/Setup/ConfigCommand.php#L98

It looks like we need a smarter check here, the warning should be removed, and the features override check should be uncommented (since this was fixed here: #2467)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant