Revert "addressing #3805: remove deprecated function call 3"

[prabhat00155]

Reverts #3861

[fmassa]

@bjuncek we got internal failures with this PR.

Prabhat will be sending a follow-up PR that addresses the issues

[fmassa]

@prabhat00155 Just a remainder that you forgot to add the [FBcode->GH] tag, as we won't be merging the other PR in fbcode so we will need to skip this one

[prabhat00155]

@prabhat00155 Just a remainder that you forgot to add the [FBcode->GH] tag, as we won't be merging the other PR in fbcode so we will need to skip this one

@fmassa Isn't [FBcode->GH] to be added when we import from FBcode?

[NicolasHug]

Isn't [FBcode->GH] to be added when we import from FBcode?

It needs to be added when we want to ignore a PR during the GH -> fbcode syncing. PRs imported from fbcode are a subset of that only. But you do make a point that the tag isn't as general as it could ;)

[prabhat00155]

Isn't [FBcode->GH] to be added when we import from FBcode?

It needs to be added when we want to ignore a PR during the GH -> fbcode syncing. PRs imported from fbcode are a subset of that only. But you do make a point that the tag isn't as general as it could ;)

@NicolasHug Thanks for explaining that. Perhaps we should have a tag called FBCode Ignore or something like that.

[datumbox]

@prabhat00155 There is another param you can use on the script to ignore a specific commit id. Just note it operates on full git hashes.