You shouldn't include entities with global.ignore tag in any @e selector. Example: check.mcfunction, line 4, if entity @e[type=minecraft:item, nbt={Item:{id:"minecraft:golden_carrot"}}
You are not going to work on the mobs to convert them, you are going to kill them, so you shouldn't include also mobs with global.ignore.kill tag or do not kill them using the tp @s ~ ~-260 ~
takeone.mcfunction. Also here you are killing an item without global.ignore tag but that could have global.ignore.kill tag.
For the rest, everything seems ok.
Project Page: https://github.com/TheSaltyPug/Mob-Conversion/releases/
Small datapack that adds conversions for:
Technical stuff