Skip to content
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

fix: Missing comma in api_build_test.py #2946

Closed
wants to merge 2 commits into from
Closed

Conversation

mrshu
Copy link

@mrshu mrshu commented Jan 29, 2022

  • The comma has been most probably left out unintentionally, leading to string concatenation between the two consecutive lines. It has been found automatically using a regular expression.

* The comma has been most probably left out unintentionally, leading to string concatenation between the two consecutive lines. It has been found automatically using a regular expression.
@rumpl
Copy link
Member

rumpl commented Feb 8, 2022

@mrshu Thank you for opening this PR, could you sign your commit please? For more information see here

@mrshu
Copy link
Author

mrshu commented Feb 8, 2022

@rumpl Should be done now 🙂

* The comma has been most probably left out unintentionally, leading to string
  concatenation between the two consecutive lines. It has been found
  automatically using a regular expression.

Signed-off-by: mr.Shu <mr@shu.io>
@milas
Copy link
Member

milas commented Jul 26, 2022

Thanks for the fix! Unfortunately, one of the commits (ca702c5) is still missing the DCO. The instructions from the bot should explain how to rebase and add it.

@milas milas added the dco/no label Aug 12, 2022
@milas
Copy link
Member

milas commented Aug 12, 2022

Feel free to re-open after updating with DCO if desired. Thanks!

@milas milas closed this Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants