Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst)
Here are some useful points:

• Pay attention to the quality of your code (flake8, mypy and type annotations). Our pre-commits will help you with that.
• In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example DAG that shows how users should use it.
• Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
• Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
• Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
• Be sure to read the Airflow Coding style.
  Apache Airflow is a community-driven project and together we are making it better 🚀.
  In case of doubts contact the developers at:
  Mailing List: [email protected]
  Slack: https://s.apache.org/airflow-slack

I think we can just remove timeout parameter.

The parameter is deprecated. Next release is a breaking change one so isn't it better to remove this parameter and adjust the fix accordingly?

I think we can just remove timeout parameter.

airflow/airflow/providers/ssh/operators/ssh.py

Lines 102 to 109 in 64174ce

	if self.timeout:
	warnings.warn(
	"Parameter `timeout` is deprecated."
	"Please use `conn_timeout` and `cmd_timeout` instead."
	"The old option `timeout` will be removed in a future version.",
	DeprecationWarning,
	stacklevel=2,
	)

The parameter is deprecated. Next release is a breaking change one so isn't it better to remove this parameter and adjust the fix accordingly?

Sure i can look into that

I think we can just remove timeout parameter.

airflow/airflow/providers/ssh/operators/ssh.py

Lines 102 to 109 in 64174ce

	if self.timeout:
	warnings.warn(
	"Parameter `timeout` is deprecated."
	"Please use `conn_timeout` and `cmd_timeout` instead."
	"The old option `timeout` will be removed in a future version.",
	DeprecationWarning,
	stacklevel=2,
	)

The parameter is deprecated. Next release is a breaking change one so isn't it better to remove this parameter and adjust the fix accordingly?

Sure i can look into that

Done!

@eladkal @potiuk Can one of you please approve so we can merge it?

Awesome work, congrats on your first merged pull request!