Dependabot adjustments

[mshima]

• PR branchs considers target branch.
• On conditions don’t show at actions list.
• dependabot builds are duplicating at dependabot/* branch and PR

---

• Please make sure the below checklist is followed for Pull Requests.

• All continuous integration tests are green

• Tests are added where necessary

• Documentation is added/updated where necessary

• Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

[SudharakaP]

@mshima : This PR seems to handle dependabot CI skipping in an alternative manner; not using the if condition as I did in #11988. I would suggest using one method to be consistent. As such,

How about we change all files angular.yml, gradle-wrapper-validation-yml, react.yml and webflux.yml to have,

on:
    push:
        branches-ignore:
            - 'dependabot/**'
on:
    pull_request:
        branches-ignore:
            - 'dependabot/**'

And remove the if conditions. wdyt? 😄

[SudharakaP]

Also my suggestion above will skip the branches when merged to master too; but do we really need to have the CI running on master for something we don't check before merging. I would suggest that's not quite optimal right? After all we should probably check what we check before merging not after merging. 😄

[mshima]

And remove the if conditions. wdyt? 😄

on:
    pull_request:
        branches-ignore:
            - 'dependabot/**'

Doesn’t work branches-ignore matches target branch only, should be master in this case.

[SudharakaP]

And remove the if conditions. wdyt? smile

on:
    pull_request:
        branches-ignore:
            - 'dependabot/**'

Doesn’t work branches-ignore matches target branch only, should be master in this case.

@mshima : Ah, got it. This is the same case with Azure which I got to know later. 😄