Skip to content

Cleaning kendra-docs-index.yaml #25

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

jwdinkel
Copy link

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