Ts-salobj

------

Changes:

* `BaseCscTestCase`: add ``timeout`` argument to ``check_bin_script``.
* Stop using the abandoned ``asynctest`` library.
* Update test function `modify_environ` to use `unittest.mock.patch` and use it in all tests
that modify os.environ (except we still don't reset env var ``LSST_DDS_PARTITION_PREFIX``
after calling `set_random_lsst_dds_partition_prefix`, which is a potential issue).
* `SalInfo`: remove read conditions from the contained dds WaitSet when closing.
ADLink suggested doing this (in my case 00020504) to avoid spurious error messages at shutdown.
* `topics.RemoteCommand`: fix a documentation error and improve the documentation
for the ``wait_done`` argument to the ``start``, ``set_start``, and ``next_ackcmd`` methods.
* `BaseCsc` and `CscCommander`: improve the documentation
for the ``index`` argument to the ``amain`` and ``make_from_cmd_line`` class methods.

Requirements:

* ts_ddsconfig
* ts_idl 2
* ts_xml 6.2
* IDL files for Test and Script generated by ts_sal 5
* SALPY_Test generated by ts_sal 5

------

Deprecations:

* Deprecate `BaseCsc.set_simulation_mode`. Note that `BaseCsc.implement_simulation_mode`,
and allowing ``valid_simulation_modes = None`` have both been deprecated for some time.
Please move all simulation mode handling to the constructor (if synchronous) or `BaseCsc.start` (if not).
* Deprecate omitting the ``version`` class attribute of CSCs.
* Deprecate `ConfigurableCsc` constructor argument ``schema_path``; please specify ``config_schema`` instead.

Changes:

* `BaseCsc`: support better help for the ``--simulate`` command-line argument,
via a new ``simulation_help`` class variable which defaults to `None`.
If not `None` and the CSC supports simulation, use this variable as the help string
for the ``--simulate`` command-line argument.
* `BaseCsc`: set the simulation mode attribute in the constructor,
instead of waiting until partway through the ``start`` method.
Warning: if ``valid_simulation_modes`` is None then we cannot check it first, but should be checked later.
* `BaseCsc`: if there is no ``version`` attribute,
set the ``cscVersions`` field of the ``softwareVersions`` event to "?",
instead of "" (that was a bug), and issue a deprecation warning.
* `ConfigurableCsc`: add constructor argument ``config_schema``.
this is the preferred way to specify the configuration schema because it allows the schema to be code,
which simplifies packaging and distribution.
* `BaseConfigTestCase`: added argument ``schema_name`` to ``check_standard_config_files``
and made ``sal_name`` optional.
* Update test for warnings to include testing for the correct message.
This makes sure the correct warning is seen (or not seen).

Requirements:

* ts_ddsconfig
* ts_idl 2
* ts_xml 6.2
* IDL files for Test and Script generated by ts_sal 5
* SALPY_Test generated by ts_sal 5

------

Changes:

* Remove test_no_commands from test_sal_info.py because ts_xml 8 no longer has a SAL component with no commands.
This makes ts_salobj compatible with bohth ts_xml 7.1 and 8.
* Update doc/conf.py to work with documenteer 0.6.

Requirements:

* ts_ddsconfig
* ts_idl 2
* ts_xml 6.2
* IDL files for Test and Script generated by ts_sal 5
* SALPY_Test generated by ts_sal 5

------

Changes:

* Add ``noarch: generic`` to the ``build`` section of ``conda/meta.yaml``.

Requirements:

* ts_ddsconfig
* ts_idl 2
* ts_xml 6.2
* IDL files for Test, Script, and LOVE generated by ts_sal 5
* SALPY_Test generated by ts_sal 5

------

Changes:

* `CscCommander`: add a digits argument to telemetry_callback method.
* Documentation: document that configuration label names must be valid python identifiers,
and must not begin with ``_`` (underscore).

Requirements:

* ts_ddsconfig
* ts_idl 2
* ts_xml 6.2
* IDL files for Test, Script, and LOVE generated by ts_sal 5
* SALPY_Test generated by ts_sal 5

------

Changes:

* Added context manager `modify_environ` to temporarily modify environment variables in unit tests.
This is rather heavyweight (it copies `os.environ`), so I don't recommended it for production code.
* `BaseScript`: modified the constructor to restore the original value (or lack of value)
of environment variable ``OSPL_MASTER_PRIORITY``, after setting it to 0 to build the `Domain`.
* `AsyncS3Bucket`: simplified to not temporarily set environment variables holding ASW S3 secrets in mock mode.
It turns out the ``moto`` mocking system already does this (and I added a test to verify that).
* `BaseCsc`: improved the output of ``_do_change_state`` to avoid an unnecessary traceback
if the called code raises `ExpectedError`.

Requirements:

* ts_ddsconfig
* ts_idl 2
* ts_xml 6.2
* IDL files for Test, Script, and LOVE generated by ts_sal 5
* SALPY_Test generated by ts_sal 5