Skip to content
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

XS✔ ◾ Commenting Code - Place 2nd half of split comment on separate line #8431

Merged
merged 3 commits into from
Apr 24, 2024

Conversation

ncn-ssw
Copy link
Member

@ncn-ssw ncn-ssw commented Apr 24, 2024

  1. What triggered this change? (PBI link, Email Subject, conversation + reason, etc)

Noticed issue while completing SugarLearning item "📋 Commenting Code"

  1. What was changed?
  • A newline was added before a set of '//', which should have been on a separate line
  • Removed unnecessary newlines above code snippet
  1. Did you do pair or mob programming (list names)?

❌ No

@ncn-ssw ncn-ssw requested a review from Geordie88 April 24, 2024 06:43
@github-actions github-actions bot added the Age: 🥚 - New About 2 hours old label Apr 24, 2024
Copy link
Contributor

github-actions bot commented Apr 24, 2024

PR Metrics

Thanks for keeping your pull request small.
Thanks for adding tests.

Lines
Product Code -
Test Code -
Subtotal -
Ignored Code 7
Total 7

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

@github-actions github-actions bot changed the title Commenting Code - Place 2nd half of split comment on separate line XS✔ ◾ Commenting Code - Place 2nd half of split comment on separate line Apr 24, 2024
@github-actions github-actions bot added Age: 🐣 - Young About 4 hours old Age: 🐥 - Adolescent About 8 hours old Age: 🐤 - Mature About 16 hours old and removed Age: 🥚 - New About 2 hours old Age: 🐣 - Young About 4 hours old Age: 🐥 - Adolescent About 8 hours old labels Apr 24, 2024
@tiagov8 tiagov8 merged commit bcff48f into main Apr 24, 2024
5 checks passed
@tiagov8 tiagov8 deleted the ncn-comments-debug-statements branch April 24, 2024 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Age: 🐤 - Mature About 16 hours old
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants