aws-cloudformation / aws-cloudformation/cloudformation-cli-python-plugin

Improve docstrings for Hooks

オープン
#269 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
107
フォーク
46
PR マージ指標
30日以内にマージされた PR はありません

説明

Hi there :wave:,

It would be nice to have Python docstrings included on the public interfaces when developing hooks.
Would pull requests for docstrings be welcome?

For example, after `cfn init` for hooks, we have a `../handlers.py` that looks similar to this:
![image](https://github.com/aws-cloudformation/cloudformation-cli-python-plugin/assets/2934552/9f35d4e1-a9bf-4ffd-840a-1b4dcef4a5bf)

In Visual Studio Code, when I hover over `cloudformation_cli_python_lib` specific classes/properties I don't get any information for what they are, or what to expect.

I couldn't find out what all the `request.hookContext` properties mean, and what `dict` keys are available on `request.hookContext.targetModel` from the [Implement Hook handlers (Python)](https://docs.aws.amazon.com/cloudformation-cli/latest/hooks-userguide/hooks-model.html#model-hook-project-code-handler-python) documentation.

In the case of `HookContext` some docstrings like this would be welcome:
![image](https://github.com/aws-cloudformation/cloudformation-cli-python-plugin/assets/2934552/4aaa6841-b274-4c36-9f1b-07d5b84f6770)

Cheers,
Neil

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。