Skip to content

Updated code comments for clarity#1519

Merged
ayush3797 merged 2 commits intomainfrom
dev/seantleonard/mappingConfigHandlingPolicy
May 29, 2023
Merged

Updated code comments for clarity#1519
ayush3797 merged 2 commits intomainfrom
dev/seantleonard/mappingConfigHandlingPolicy

Conversation

@seantleonard
Copy link
Copy Markdown
Contributor

Why make this change?

  • While evaluating another bug, came across errors and lack of clarity in some comments. I went ahead and made some of those changes.

What is this change?

  • Comment updates. No functional changes.
  • Only code changes were to move functions to one line, which weren't long enough to justify splitting up across multiple lines.

How was this tested?

N/A no functional changes.

@ayush3797 ayush3797 enabled auto-merge (squash) May 29, 2023 07:09
@ayush3797 ayush3797 merged commit 4717b1b into main May 29, 2023
@ayush3797 ayush3797 deleted the dev/seantleonard/mappingConfigHandlingPolicy branch May 29, 2023 07:31
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.

3 participants