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

DT-981: Fixes #3921: Enabled D9 deprecated code scanning by default. #3946

Merged
merged 1 commit into from
Dec 12, 2019

Conversation

danepowell
Copy link
Contributor

@danepowell danepowell commented Dec 10, 2019

Fixes #3921

Changes proposed

  • Enable deprecated code scanning of custom modules and themes by default. Note that this will not apply if a user has manually overridden this via blt.yml.

Steps to replicate the issue

  1. Run blt validate
  2. Ensure that tests:deprecated:modules and tests:deprecated:themes are run automatically as part of the tests.

@danepowell danepowell added Enhancement A feature or feature request Major change Requires change record (often BC-breaking) 11.x Affects or applies to 11.x labels Dec 10, 2019
@danepowell danepowell merged commit 15f2ae7 into acquia:11.x Dec 12, 2019
@jedgar1mx
Copy link

This is throwing an error during my build, is there a way to temporary bypass them?
THanks

@danepowell
Copy link
Contributor Author

@jedgar1mx the process for disabling scanning is described in the BLT 11 release notes.

If you read that already and it wasn't clear, let me know how we could improve it.

If you haven't read that already, you really should 😄

@jedgar1mx
Copy link

jedgar1mx commented Dec 18, 2019 via email

@Luukyb
Copy link

Luukyb commented Jun 23, 2020

Hi,
I arrived on this PR from the release notes of BLT 11.x. It was a bit difficult for me to find out how to disable the depreciation tests. I first was looking at "disable-targets" but it has to be in "validate". I will just share what I added in my blt.yml if that can help someone else:

validate:
  deprecation:
    modules: false
    themes: false

Thanks,
Luc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11.x Affects or applies to 11.x Enhancement A feature or feature request Major change Requires change record (often BC-breaking)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DT-981: Enable Drupal Check (D9 Deprecation) By Default
5 participants