Skip to content

fix: better api annotation of valcons matching selection's type#1071

Merged
georgefst merged 1 commit intomainfrom
brprice/valcon-matches-type
Jun 11, 2023
Merged

fix: better api annotation of valcons matching selection's type#1071
georgefst merged 1 commit intomainfrom
brprice/valcon-matches-type

Conversation

@brprice
Copy link
Contributor

@brprice brprice commented Jun 11, 2023

No description provided.

@brprice brprice requested a review from a team June 11, 2023 11:43
@brprice
Copy link
Contributor Author

brprice commented Jun 11, 2023

This implements #1067 (comment)

Copy link
Contributor

@georgefst georgefst 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!

Signed-off-by: Ben Price <ben@hackworthltd.com>
@brprice brprice force-pushed the brprice/valcon-matches-type branch from a771fe1 to edd18ce Compare June 11, 2023 11:47
@georgefst georgefst enabled auto-merge June 11, 2023 11:49
@dhess
Copy link
Member

dhess commented Jun 11, 2023

Thanks! However, keep in mind that what's currently deployed is not main but #1070, so maybe add this to that PR so we can get it online and running?

@georgefst georgefst added this pull request to the merge queue Jun 11, 2023
@dhess
Copy link
Member

dhess commented Jun 11, 2023

I will rebase #1070 when this PR is merged to main, so that we can get this deployed. Thanks all!

Merged via the queue into main with commit 4e36f4c Jun 11, 2023
@georgefst georgefst deleted the brprice/valcon-matches-type branch June 11, 2023 12:16
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.

3 participants