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

add Protocol Version to single condition field types #222

Merged
merged 3 commits into from
Jul 8, 2024

Conversation

dbtlabs-aaron
Copy link
Contributor

The Protocol Version is a single condition field type. Plans with rules using this condition will throw a warning when this ought to be a supported type.

The Protocol Version is a single condition field type. Plans with rules using this condition will throw a warning when this ought to be a supported type.
@daniel-corbett
Copy link
Collaborator

Hi @dbtlabs-aaron - nice catch! Thanks for the contribution. Can you please run make docs as well to autogenerate the related documentation? Thanks!

@daniel-corbett daniel-corbett self-requested a review May 31, 2024 15:18
@dbtlabs-aaron
Copy link
Contributor Author

docs updated @daniel-corbett

@daniel-corbett
Copy link
Collaborator

@dbtlabs-aaron - seems your latest commit removed one of our guides docs/guides/3.0.0.md. Please undo that and we'll be good to go.

@dbtlabs-aaron
Copy link
Contributor Author

Hoping that change does it @daniel-corbett. It looks like the make docs routine deleted it.

@daniel-corbett
Copy link
Collaborator

Looks good. Thanks @dbtlabs-aaron !

@daniel-corbett daniel-corbett merged commit f458b71 into signalsciences:main Jul 8, 2024
2 checks passed
@dbtlabs-aaron dbtlabs-aaron deleted the patch-1 branch July 9, 2024 18:12
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