Restrict init file gen to package directories and fix docstring blank lines

[arandito]

Description

This PR removes __init__.py file generation in non-package directories such as docs/*.

Examples

Bedrock Runtime

Before:

docs
├── client
│   ├── __init__.py
│   ├── apply_guardrail.rst
│   └── ...
├── models
│   ├── __init__.py
│   ├── AccessDeniedException.rst
│   └── ...

After:

docs
├── client
│   ├── apply_guardrail.rst
│   └── ...
├── models
│   ├── AccessDeniedException.rst
│   └── ...

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[nateprewitt]

This seems a little strange for the fix. Is it because this adds the newline we need but our actual writer is also always appending a newline so we get two? I think this issue was affecting multiple places. I wonder if we should be doing something more like trimming extra newlines from the end of the final docstring (or not appending them to the end).

[arandito]

Yes there are a couple places where we add two newlines on accident by using write instead of writeInline and then passing in a text block which adds its own newline.

I pushed up a commit that adds a helper method to PythonWriter that removes all trailing whitespaces. We'll call it and then add a single newline before writing the closing """ in writeDocs(). This normalizes all docstrings and prevents a trailing blank line.

This fixed all of the issues I've seen in models.py and client.py. I also made the ConfigGenerator use writeDocs() (it didn't previously) and it resolved the blank line issue for its constructor as well.

I still fixed the source of the double newlines but the helper method will help prevent this issue from reoccurring in the future.

This is the diff for the Transcribe client with these codegen changes.

[nateprewitt]

Looks good, thanks @arandito!