Skip to content

Document more source types v2 - #610

Open
jorbaum wants to merge 5 commits into
cloudfoundry:masterfrom
jorbaum:document-more-source-types-v2
Open

Document more source types v2#610
jorbaum wants to merge 5 commits into
cloudfoundry:masterfrom
jorbaum:document-more-source-types-v2

Conversation

@jorbaum

@jorbaum jorbaum commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Follow up of #577 / #609 .

@jorbaum
jorbaum force-pushed the document-more-source-types-v2 branch from fa0559a to b74ae11 Compare August 20, 2026 07:47
I took a deeper look to double check and adjusted as per my findings
@jorbaum
jorbaum force-pushed the document-more-source-types-v2 branch from b74ae11 to 3c71eb1 Compare August 20, 2026 07:48

@chombium chombium left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@jorbaum Nice improvements, but I have few comments.

Comment thread deploy-apps/streaming-logs.html.md.erb Outdated
Comment thread deploy-apps/streaming-logs.html.md.erb Outdated
Comment thread deploy-apps/streaming-logs.html.md.erb
Comment thread deploy-apps/streaming-logs.html.md.erb
@jorbaum

jorbaum commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

@chombium thanks for the review. I worked in most of your comments. Marking it ready for review now even though I left the one comment about the newline unresolved.

@jorbaum
jorbaum marked this pull request as ready for review August 21, 2026 09:26
Comment thread deploy-apps/streaming-logs.html.md.erb Outdated
Comment thread deploy-apps/streaming-logs.html.md.erb Outdated
@jorbaum
jorbaum force-pushed the document-more-source-types-v2 branch from 9f20694 to ddfde5b Compare August 21, 2026 13:50

@chombium chombium left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lgtm. Thanks @jorbaum

@chombium chombium left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lgtm. Thanks @jorbaum

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