Skip to content

Cleaning kendra-docs-index.yaml#25

Open
jwdinkel wants to merge 1 commit into
aws-samples:developmentfrom
jwdinkel:patch-2
Open

Cleaning kendra-docs-index.yaml#25
jwdinkel wants to merge 1 commit into
aws-samples:developmentfrom
jwdinkel:patch-2

Conversation

@jwdinkel
Copy link
Copy Markdown

Fixed python import and Use Sub instead of Join to make code more readable.

Issue #, if available:

Description of changes:
I have removed the import of random, which was not used.
I have also changed code to use inline variables rather than the multi-line syntax that was used before with Sub and with Join. This reduces the code by 30 lines and makes it look much cleaner in code editors that highlight the variables. In addition, the Type fields are consistent now with quotes rather than having just some that are quoted in the prior version. And I corrected a small grammar error in the comment.

Hopefully, you will agree, this is much cleaner.

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

Simplified and aligned syntax. Fixed Python imports.
@MithilShah MithilShah changed the base branch from main to development August 14, 2023 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant