Skip to content

#130 - Fixes issue with nested --- in package metadata #131

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

Titou325
Copy link
Contributor

@Titou325 Titou325 commented Nov 7, 2023

Implements a small fix regarding issues with nested --- in package metadata.

Fixes #130

Copy link
Contributor

@varungandhi-src varungandhi-src left a comment

Choose a reason for hiding this comment

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

Thanks!

@varungandhi-src
Copy link
Contributor

Super minor thing: since the "Fixes BLAH" bit in the PR description automatically associates the issue with the right PR, I would avoid specifying the issue again in the commit or PR title, since that creates more visual noise on the issue page. For example, the commit message line keeps showing up repeatedly in the issue on enclosing ranges, which just requires more scrolling without adding anything.

CleanShot 2023-11-17 at 11 50 50@2x

@varungandhi-src varungandhi-src merged commit f5e9aee into sourcegraph:scip Nov 17, 2023
@varungandhi-src
Copy link
Contributor

Oops, I goofed up following my own advice by not properly editing the commit title when merging this PR 🙈 f5e9aee

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.

Automatic package detection fails if any field contains ---
2 participants