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.

Sign up for GitHub

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

Jump to bottom

Prepare for 2023.3: Do not depend on MacroManager.getInstance() #5390

Merged
merged 1 commit into from
Sep 26, 2023
Merged

Prepare for 2023.3: Do not depend on MacroManager.getInstance() #5390

merged 1 commit into from
Sep 26, 2023

Commits on Sep 26, 2023

  1. Prepare for 2023.3: Do not depend on MacroManager.getInstance() 

    Probably caused by this change
JetBrains/intellij-community@b3ed617#diff-737d7a5e2b283fc8b2a9c7f301e689e29a83629c1a3533053b813570da2e59f4R21

Before that, MacroManager was instantiated only if it was present in LangExtensions.xml (not a thing in Bazel Plugin tests).
Now, it is instantiated always, which results in a crash in expandMacros call in tests. Hence, we need a new way to disable
macro expansion in tests.
    @tpasternak
    tpasternak committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    4f6c1a0 View commit details
    Browse the repository at this point in the history