Calling Developers!
We are reenergizing our code contribution process! Learn More

What are the Slack Archives?

It’s a history of our time together in the Slack Community! There’s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you won’t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

Hi everyone, is there a way to disable a certain architecture sniffer rule for one method? Im tryin

Options
U01BV4LNUKZ
U01BV4LNUKZ Posts: 9 🧑🏻‍🚀 - Cadet

Hi everyone,

is there a way to disable a certain architecture sniffer rule for one method? Im trying to use

@SuppressWarnings(FactoryOnlyGetAndCreateRule)

but the sniffer does still complain.

Comments

  • U01BV4LNUKZ
    U01BV4LNUKZ Posts: 9 🧑🏻‍🚀 - Cadet
    Options

    It is working on class level though, but I do not want to suppress to future issues.

  • USZ0XG6SK
    USZ0XG6SK Posts: 111 🧑🏻‍🚀 - Cadet
    Options

    just of curiosity, why do you want to do this?

  • U01BV4LNUKZ
    U01BV4LNUKZ Posts: 9 🧑🏻‍🚀 - Cadet
    Options

    I'm within DataImportBusinessFactory and we are creating a quite complex importer with multiple step broker and steps that is used by more than one importer (same steps for two sources of data).

    so we moved the redundand code to a setup(DataSetStepBrokerAwareInterface $dataImporter) method, and this name is not allowed. 😞

  • U01SE4SRCU9
    U01SE4SRCU9 Posts: 68 🧑🏻‍🚀 - Cadet
    Options

    @U01BV4LNUKZ Then why not just rename that method to something like createCoreSteps, createSetupSteps, etc.?