Hi, please tell me who will merge it, I can't do it myself.

@dmurphy18 I don't have the right to merge it, can you merge?

@sakateka I don't have merge rights either, but just re-reviewing this, QQ is the change already covered by existing tests ?, if not, it should have a test to show the fix is working.

change already covered by existing tests ?

This is just removing dead code, there is no impact on existing code.

Updated PR with current master branch and re-running tests, if no fails will push to get it in - got to fix those leaks

@sakateka Can you look at the failing tests on Ubuntu 22.04. I believe this might be due to directory changes on Ubuntu 22.04 from /usr/share to /etc/api. Take a look at @Ch3LL PR #62159, so might have to wait till this goes in and then re-run tests

!merge

4 approvals - It's very cool, but I can't merge, I don't have rights

Hi! I'm your friendly PR bot!

You might be wondering what I'm doing commenting here on your PR.

Yes, as a matter of fact, I am...

I'm just here to help us improve the documentation. I can't respond to
questions or anything, but what I can do, I do well!

Okay... so what do you do?

I detect modules that are missing docstrings or "CLI Example" on existing docstrings!
When I was created we had a lot of these. The documentation for these
modules need some love and attention to make Salt better for our users.

So what does that have to do with my PR?

I noticed that in this PR there are some files changed that have some of these
issues. So I'm leaving this comment to let you know your options.

Okay, what are they?

Well, my favorite, is that since you were making changes here I'm hoping that
you would be the most familiar with this module and be able to add some other
examples or fix any of the reported issues.

If I can, then what?

Well, you can either add them to this PR or add them to another PR. Either way is fine!

Well... what if I can't, or don't want to?

That's also fine! We appreciate all contributions to the Salt Project. If you
can't add those other examples, either because you're too busy, or unfamiliar,
or you just aren't interested, we still appreciate the contributions that
you've made already.

Whatever approach you decide to take, just drop a comment here letting us know!

Check Known Missing Docstrings...........................................Failed
- hook id: invoke
- duration: 1.24s
- exit code: 1
/home/runner/.cache/pre-commit/repo_kavnz8y/py_env-python3/lib/python3.9/site-packages/_distutils_hack/init.py:30: UserWarning: Setuptools is replacing distutils.

warnings.warn("Setuptools is replacing distutils.")

The function 'load' on 'salt/utils/yamlloader.py' does not have a docstring

The function 'load' on 'salt/utils/yamlloader_old.py' does not have a docstring

Found 2 errors

Thanks again!

re-run pr-ubuntu-2204-amd64-py3-m2crypto-pytest
re-run pr-photon-3-x86_64-py3-pytes