Skip to content
This repository was archived by the owner on Apr 13, 2025. It is now read-only.

nodecg-io-googleapis - closes #226 #252

Merged
merged 2 commits into from
Oct 10, 2021

Conversation

LarsVomMars
Copy link
Contributor

No description provided.

@derNiklaas
Copy link
Contributor

Hi! I don't think "closes #226" would close #226 if it's in the title of the PR (I could be wrong though because I've never tried it)

@J0B10 J0B10 linked an issue Oct 8, 2021 that may be closed by this pull request
@J0B10
Copy link
Member

J0B10 commented Oct 8, 2021

It should, as the merge commit will by default contain the PR title in the commit message so it will also contain the close note, which will therefore close the issue.

The better solution is still to just write it in the PR description which triggers githubs feature that links the issue and PR.
I now did it manually.

@hlxid hlxid self-requested a review October 9, 2021 05:33
Copy link
Member

@hlxid hlxid left a comment

Choose a reason for hiding this comment

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

LGTM, just some smaller things. Thanks for your PR 😃!
Are you planning on creating a sample for this service? You can do it in a seperate PR if you want.

@LarsVomMars
Copy link
Contributor Author

Yeah, I was working on updating the available samples and fixing some issues (2 of your mentions). But didn't have the time to finish and commit it yet

@hlxid hlxid self-requested a review October 10, 2021 06:01
Copy link
Member

@hlxid hlxid left a comment

Choose a reason for hiding this comment

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

Thank you very much! 😄

@hlxid hlxid merged commit 8233238 into codeoverflow-org:master Oct 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create nodecg-io-googleapis
4 participants