Skip to content

Resolve LNDENG-4197#549

Merged
marqode merged 1 commit intomainfrom
LNDENG-4197
Apr 27, 2026
Merged

Resolve LNDENG-4197#549
marqode merged 1 commit intomainfrom
LNDENG-4197

Conversation

@marqode
Copy link
Copy Markdown
Contributor

@marqode marqode commented Apr 10, 2026

Summary

Added line wrapping to error output in ActivityDetails.tsx to make it more readable. Set wrapLines: true on CodeSnippet for mirror error output.

Release Impact

According to the Landscape Server Release Cycle, this change will target the following release cycle:

  • Target Branch: main
  • Version Impact:
    • Patch (Fix)
    • Minor (Feature)
    • Major (Breaking)

Checklist

  • Changeset Added: I have run pnpm changeset and committed the resulting .md file.
  • UI Verified: I have verified the changes locally.
  • Linting: No linting errors are present (especially in scripts/).

Versioning Reminder

Important

This repository now uses CalVer ($YY.0M.Point.Patch$).
Please ensure your changeset description is clear, as it will be automatically added to the CHANGELOG.md upon merging to main.

Copy link
Copy Markdown
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

Improves readability of activity output (notably mirror-related error/output logs) by enabling line wrapping in the CodeSnippet used by ActivityDetails.

Changes:

  • Enabled wrapLines: true on the CodeSnippet block in ActivityDetails for wrapped output rendering.
  • Added a patch changeset documenting the UI readability improvement for /repositories/mirrors.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/features/activities/components/ActivityDetails/ActivityDetails.tsx Enables wrapped lines for the activity “Output” code snippet to improve readability.
.changeset/some-dryers-bake.md Adds a patch changeset entry describing the mirror error/output wrapping change.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@rubinaga rubinaga left a comment

Choose a reason for hiding this comment

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

LGTM

@marqode marqode merged commit cf25605 into main Apr 27, 2026
10 of 11 checks passed
@marqode marqode deleted the LNDENG-4197 branch April 27, 2026 15:04
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