Skip to content

ordering of issue templates and other small improvements#429

Merged
itcarroll merged 5 commits into
mainfrom
issue-template-update-notebook
Apr 9, 2026
Merged

ordering of issue templates and other small improvements#429
itcarroll merged 5 commits into
mainfrom
issue-template-update-notebook

Conversation

@itcarroll

@itcarroll itcarroll commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Fixes #430

  • ordered
  • labels corrected and/or created
  • line breaks in markdown resolved

Really a workaround rather than a resolution on the line breaks. Not ideal for editors, but I suspect there's a problem on GitHub's side with parsing multi-line strings as raw markdown.

@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

@github-actions github-actions Bot temporarily deployed to pull request April 9, 2026 20:35 Inactive
@itcarroll itcarroll marked this pull request as draft April 9, 2026 20:36
@github-actions github-actions Bot temporarily deployed to pull request April 9, 2026 20:51 Inactive
@github-actions github-actions Bot temporarily deployed to pull request April 9, 2026 20:54 Inactive
@itcarroll itcarroll requested a review from ateucher April 9, 2026 20:59
@itcarroll itcarroll marked this pull request as ready for review April 9, 2026 20:59
@github-actions github-actions Bot temporarily deployed to pull request April 9, 2026 20:59 Inactive

@ateucher ateucher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great, thanks @itcarroll! I'll leave up to you if you'd like put the line breaks back in with the > instead of | in the multiline text fields.

@@ -42,29 +42,19 @@ body:
- type: markdown
attributes:
value: |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
value: |
value: >

Using > instead of | removes line breaks in multiline text fields; | preserves them (https://stackoverflow.com/a/72084881). Sorry I missed this the first time!


- type: markdown
attributes:
value: |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
value: |
value: >

Same as above

@@ -53,15 +53,10 @@ body:
- type: markdown
attributes:
value: |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
value: |
value: >

@itcarroll itcarroll merged commit a5ed566 into main Apr 9, 2026
3 checks passed
@github-actions github-actions Bot temporarily deployed to pull request April 9, 2026 22:53 Inactive
@itcarroll itcarroll deleted the issue-template-update-notebook branch April 10, 2026 00:58
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.

Technical problems with issue templates

2 participants