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

feat: view and edit kinds on foralls #1036

Merged
merged 2 commits into from
Sep 28, 2023
Merged

feat: view and edit kinds on foralls #1036

merged 2 commits into from
Sep 28, 2023

Conversation

brprice
Copy link
Contributor

@brprice brprice commented Aug 17, 2023

No description provided.

@brprice brprice added the Deploy service Deploy the backend service for this PR label Aug 17, 2023
@dhess dhess added Deploy service Deploy the backend service for this PR and removed Deploy service Deploy the backend service for this PR labels Aug 28, 2023
@brprice brprice force-pushed the brprice/forall-kind-edit branch from adbe00a to fe3e2f5 Compare September 12, 2023 11:44
@brprice brprice added Deploy service Deploy the backend service for this PR and removed Deploy service Deploy the backend service for this PR labels Sep 12, 2023
@brprice brprice force-pushed the brprice/forall-kind-edit branch from fe3e2f5 to 092902e Compare September 12, 2023 11:45
@brprice brprice removed the Deploy service Deploy the backend service for this PR label Sep 12, 2023
@brprice brprice added the Deploy service Deploy the backend service for this PR label Sep 12, 2023
@dhess dhess added Deploy service Deploy the backend service for this PR and removed Deploy service Deploy the backend service for this PR labels Sep 17, 2023
The new Primer pin adds a new option to EvalFull (which controls
evaluation under binders), and changes the default behaviour to not
evaluate under binders.

Signed-off-by: Ben Price <[email protected]>
This bumps the Primer pin, which adds support for higher-kinded
polymorphism. There are no other changes required to take advantage of
this feature.

Signed-off-by: Ben Price <[email protected]>
@brprice brprice force-pushed the brprice/forall-kind-edit branch from 092902e to 6b5a139 Compare September 26, 2023 16:01
@brprice brprice changed the title WIP/preview: view+edit kinds on foralls feat: view and edit kinds on foralls Sep 26, 2023
@brprice brprice marked this pull request as ready for review September 26, 2023 16:02
@brprice brprice requested a review from a team September 26, 2023 16:02
@brprice brprice added Deploy service Deploy the backend service for this PR and removed Deploy service Deploy the backend service for this PR labels Sep 26, 2023
@brprice
Copy link
Contributor Author

brprice commented Sep 26, 2023

I've done this bump in two stages, since the earlier one introduces some changes in the orval-generated model which are orthogonal to the purpose of this PR (which is contained in the second commit). Both commits are clean bumps, with no code changes required.

@brprice
Copy link
Contributor Author

brprice commented Sep 28, 2023

@dhess mentioned he wanted to take a look at this, so I'll leave it to him to merge if/when he is happy.

@dhess
Copy link
Member

dhess commented Sep 28, 2023

Given that it's literally just a pin bump (except for the upstream eval changes, which are unrelated), let's merge it, and if there are any issues down the line — and I don't expect there will be any — we'll just deal with them separately.

@dhess dhess added this pull request to the merge queue Sep 28, 2023
Merged via the queue into main with commit 565d1c3 Sep 28, 2023
8 checks passed
@dhess dhess deleted the brprice/forall-kind-edit branch September 28, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deploy service Deploy the backend service for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants