Skip to content

fix: resolve broken internal links flagged by lint#525

Merged
pru55e11 merged 2 commits intomainfrom
fix-lint-errors
Apr 10, 2026
Merged

fix: resolve broken internal links flagged by lint#525
pru55e11 merged 2 commits intomainfrom
fix-lint-errors

Conversation

@pru55e11
Copy link
Copy Markdown
Collaborator

@pru55e11 pru55e11 commented Apr 9, 2026

Summary

Fixes all missing-file lint errors that started failing after a lint tool update. All broken links are in pre-existing files — no content changes, navigation corrections only.

Changes

File Fix
troubleshoot.md application_logging.mdapplication_logging/logging.md
logging.md, architecture-overview.md, guides/index.md, resources/index.md app_builder_overview.mdwhat-is-app-builder.md (file was renamed)
guides/app_builder_guides/index.md, guides/index.md events/custom-events.md../../resources/event-driven/ (page moved to resources)
storage/application-state.md security/index.md../security/ (missing ../ in relative path)
creating-actions.md, runtime_guides/index.md, tools/index.md Removed wskdeploy_install.md references (file removed, wskdeploy is deprecated)

Related

Discovered when I created PR #524 (release notes update) which is failing lint due to these pre-existing errors.

@shazron
Copy link
Copy Markdown
Member

shazron commented Apr 10, 2026

It's still failing the lint check (4 errors)

- troubleshoot.md: correct relative path to application_logging/logging.md
- app_builder_guides/index.md, guides/index.md: add index.md to event-driven links
- application-state.md: add index.md to security link
@pru55e11
Copy link
Copy Markdown
Collaborator Author

It's still failing the lint check (4 errors)

That'll teach me to not wait for the linter to finish the run before posting the PR link

@pru55e11 pru55e11 merged commit b39262d into main Apr 10, 2026
2 checks passed
@pru55e11 pru55e11 deleted the fix-lint-errors branch April 10, 2026 17:23
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.

2 participants