Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Kocal
Copy link
Contributor

@Kocal Kocal commented Dec 3, 2022

Hi,

This is a proposal to make the feature from #207 more tolerant about mistakes when specifying branch in the Symfony's PR template.

E.g when opening symfony/symfony#48451, visually everything was fine but I forgot an HTML comment (which is visually... invisible), which made Carson a bit angry about that :P
image

Since maintainers look for the HTML-rendered table and not for the Markdown-source table, I think it could be nice to remove HTML comments when checking for the base branch.

WDYT?
Thanks

@alamirault
Copy link
Contributor

alamirault commented Dec 3, 2022

Hey @Kocal, I was on it ! #211 :)

Didn't see your PR when start mine

@weaverryan
Copy link
Contributor

Lol, and I ALSO didn't see this PR until #211. Thanks for this @Kocal - your version should have been merged, but for some reason I clicked in 211 first 🙃 . Kind of cool to see this fixed twice so quickly

@weaverryan weaverryan closed this Dec 5, 2022
@Kocal
Copy link
Contributor Author

Kocal commented Dec 5, 2022

No problem, at least this is fixed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants