Skip to content
New issue

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

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

Already on GitHub? # to your account

Hotfix/remove inline comments #535

Closed
wants to merge 3 commits into from
Closed

Conversation

aarz-snl
Copy link
Collaborator

@aarz-snl aarz-snl commented Dec 16, 2024

🗣 Description

Hotfix to remove inline comments causing issues during install.

💭 Motivation and context

Inline comment causes issues during install because of parsing problems with systemd.

🧪 Testing

Has been tested and already merged into develop. Merging to main to create a quicker release.

✅ Pre-approval checklist

  • Changes are limited to a single goal AND
    the title reflects this in a clear human readable format
  • Issue that this PR solves has been selected in the Development section
  • I have read and agree to LME's CONTRIBUTING.md document.
  • The PR adheres to LME's requirements in RELEASES.md
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated to reflect the changes in this PR.

✅ Pre-merge Checklist

  • All tests pass
  • PR has been tested and the documentation for testing is above
  • Squash and merge all commits into one PR level commit

✅ Post-merge Checklist

  • Delete the branch to keep down number of branches

@aarz-snl aarz-snl requested a review from mreeve-snl December 16, 2024 16:41
@aarz-snl aarz-snl closed this Dec 16, 2024
@aarz-snl aarz-snl deleted the hotfix/remove-inline-comments branch December 16, 2024 16:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants