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.

Sign up for GitHub

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

Jump to bottom

remove reference to unused ok to test label #5149

Merged
merged 1 commit into from
May 2, 2022
Merged

remove reference to unused ok to test label #5149

merged 1 commit into from
May 2, 2022

Conversation

emmyoop
Copy link
Member

@emmyoop emmyoop commented Apr 25, 2022 •
edited
Loading

resolves #4904

Description

Update the contributing doc to not reference the ok to test label as we no longer use it.

Checklist

@emmyoop emmyoop added the Skip Changelog Skips GHA to check for changelog file label Apr 25, 2022
@emmyoop emmyoop requested a review from jtcohen6 April 25, 2022 16:10
@emmyoop emmyoop requested a review from a team as a code owner April 25, 2022 16:10
@emmyoop emmyoop requested a review from gshank April 25, 2022 16:10
@cla-bot cla-bot bot added the cla:yes label Apr 25, 2022
@stu-k stu-k requested a review from a team April 25, 2022 16:37
@emmyoop emmyoop merged commit 935edc7 into main May 2, 2022
@emmyoop emmyoop deleted the er/contributor-commit-label branch May 2, 2022 14:21
agoblet pushed a commit to BigDataRepublic/dbt-core that referenced this pull request May 20, 2022
@emmyoop
remove reference to unused ok to test label (dbt-labs#5149)
db32289
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stu-k stu-k stu-k approved these changes

@jtcohen6 jtcohen6 Awaiting requested review from jtcohen6

@gshank gshank Awaiting requested review from gshank gshank was automatically assigned from dbt-labs/core-group

Assignees
No one assigned
Labels
cla:yes Skip Changelog Skips GHA to check for changelog file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CT-388] NO ISSUE
2 participants
@emmyoop @stu-k