-
Notifications
You must be signed in to change notification settings - Fork 75
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
OSS standardization changes #474
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Howdy! Thank you so much for doing this work 🙂 I think we have some lines from contributing template that aren't applicable to this repository
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Ship it! Thank you!
CONTRIBUTING.md
Outdated
|
||
Once all tests are passing and your PR has been approved, a `dbt-docs` maintainer will merge your changes into the active development branch. And that's it! Happy developing :tada: | ||
|
||
Sometimes, the content license agreement auto-check bot doesn't find a user's entry in its roster. If you need to force a rerun, add `@cla-bot check` in a comment on the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same about possibly adding a troubleshooting header. not blocking
attributes: | ||
label: Context | ||
description: | | ||
Provide the "why", motivation, and alternative approaches considered -- linking to previous refinement issues, spikes, Notion docs as appropriate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spikes, docs as appropriate
Worth dropping the Notion reference since that is an implementation detail? Then if we ever moved off of Notion we wouldn't have to update this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left non-blocking comments
resolves dbt-labs/dbt-core#9221
Description
add contributing doc and dependabot alerts, modify readme to bring this OSS repo up to the minimal standards for OSS at dbt Labs.
I am unsure of the impact turning on dependabot will have. We will need to revisit once it's been on and we see what it generates.
Checklist
changie new
to create a changelog entry