Skip to content

Comments

Remove Symlink + Log header location#1020

Merged
Romanitho merged 2 commits intodevelopfrom
remove-symlink
Aug 11, 2025
Merged

Remove Symlink + Log header location#1020
Romanitho merged 2 commits intodevelopfrom
remove-symlink

Conversation

@Romanitho
Copy link
Owner

Proposed Changes

  • Remove symlink as not working anymore
  • Move back log header on top

Copilot AI review requested due to automatic review settings August 11, 2025 16:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes non-functional symlink creation code and reorganizes log header initialization to improve script structure and maintainability.

  • Removes symlink creation logic for Intune Management Extension logs as it's no longer working
  • Moves WAU configuration reading and winget source setup after the log header initialization

@github-actions
Copy link
Contributor

github-actions bot commented Aug 11, 2025

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Warnings Elapsed time
⚠️ COPYPASTE jscpd yes 3 no 2.31s
⚠️ MARKDOWN markdownlint 3 0 2 0 0.91s
✅ MARKDOWN markdown-link-check 3 0 0 5.38s
✅ MARKDOWN markdown-table-formatter 3 0 0 0 0.26s
✅ POWERSHELL powershell 4 0 0 8.19s
✅ POWERSHELL powershell_formatter 4 0 0 5.8s
⚠️ REPOSITORY checkov yes 3 no 12.31s
⚠️ REPOSITORY devskim yes 1 8 1.46s
✅ REPOSITORY dustilock yes no no 0.01s
✅ REPOSITORY gitleaks yes no no 0.58s
✅ REPOSITORY git_diff yes no no 0.01s
✅ REPOSITORY grype yes no no 24.03s
✅ REPOSITORY kics yes no no 1.23s
✅ REPOSITORY secretlint yes no no 0.5s
✅ REPOSITORY syft yes no no 1.2s
✅ REPOSITORY trivy yes no no 4.83s
✅ REPOSITORY trivy-sbom yes no no 0.1s
✅ REPOSITORY trufflehog yes no no 2.29s
⚠️ SPELL cspell 8 27 0 5.05s
✅ SPELL lychee 3 0 0 0.99s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@Romanitho
Copy link
Owner Author

#1018

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Romanitho Romanitho merged commit 416b780 into develop Aug 11, 2025
2 checks passed
@Romanitho Romanitho deleted the remove-symlink branch August 11, 2025 20:32
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