Skip to content

[PECO-247] Add DCO Check#19

Merged
susodapop merged 3 commits into
mainfrom
add-dco
Aug 24, 2022
Merged

[PECO-247] Add DCO Check#19
susodapop merged 3 commits into
mainfrom
add-dco

Conversation

@susodapop
Copy link
Copy Markdown

Mirrors the changes to NodeJS and Python connectors.

Signed-off-by: Jesse Whitehouse <jesse@whitehouse.dev>
@github-actions
Copy link
Copy Markdown

Thanks for your contribution! To satisfy the DCO policy in our contributing guide every commit message must include a sign-off message. One or more of your commits is missing this message. You can reword previous commit messages with an interactive rebase (git rebase -i main).

Signed-off-by: Jesse Whitehouse <jesse@whitehouse.dev>
Comment thread CONTRIBUTING.md Outdated
# Contributing Guide

To contribute to this repository, fork it and send pull requests.
We happily welcome contributions to this package. We use [GitHub Issues](https://github.com/databricks/databricks-sql-nodejs/issues) to track community reported issues and [GitHub Pull Requests](https://github.com/databricks/databricks-sql-nodejs/pulls) for accepting changes.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

one small issue, the link is to nodejs not go?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed in 4cc8942

Signed-off-by: Jesse Whitehouse <jesse@whitehouse.dev>
@susodapop susodapop merged commit 30ea290 into main Aug 24, 2022
@susodapop susodapop deleted the add-dco branch August 24, 2022 17:15
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.

2 participants