Skip to content

Conversation

@salif
Copy link
Contributor

@salif salif commented Aug 26, 2023

Please confirm this pull request meets the following requirements:

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Editing an existing topic or collection

I'm suggesting these edits to an existing topic or collection:

  • Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
  • Content (and my changes are in index.md)

vlang/v is already on line 50 in the same file, link


Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

@salif salif requested a review from a team as a code owner August 26, 2023 12:43
Copy link
Contributor

@kenyonj kenyonj left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks for your contribution!

@kenyonj kenyonj enabled auto-merge August 29, 2023 14:00
@ahpook ahpook disabled auto-merge September 8, 2023 21:17
@ahpook
Copy link
Contributor

ahpook commented Sep 8, 2023

The test didn't catch this case.

@ahpook ahpook merged commit f0244eb into github:main Sep 8, 2023
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