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

[query] Move is.hail.check to test code #14801

Open
wants to merge 1 commit into
base: ehigham/ctx-coercer-cache
Choose a base branch
from

Conversation

ehigham
Copy link
Member

@ehigham ehigham commented Jan 24, 2025

Change Description

Fixes #<issue_number> (delete if N/A)

Brief description and justification of what this PR is doing.

Security Assessment

Delete all except the correct answer:

  • This change has a high security impact
    • Required: The impact has been assessed and approved by appsec
  • This change has a medium security impact
  • This change has a low security impact
  • This change has no security impact

Impact Description

For none/low impact: a quick one/two sentence justification of the rating.

  • Example: "Docs only", "Low-level refactoring of non-security code", etc.
    For medium/high impact: provide a description of the impact and the mitigations in place.
  • Example: "New UI text field added in analogy to existing elements, with input strings escaped and validated against code injection"

(Reviewers: please confirm the security impact before approving)

@ehigham ehigham marked this pull request as ready for review January 24, 2025 20:36
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from df36f36 to 51edeb7 Compare January 24, 2025 20:54
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from b6a31f7 to fc08ea1 Compare January 24, 2025 20:54
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from 51edeb7 to 07eaf29 Compare January 27, 2025 23:15
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from fc08ea1 to 97dd229 Compare January 27, 2025 23:15
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch 2 times, most recently from 006475e to 8330580 Compare January 29, 2025 20:22
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from 97dd229 to db88046 Compare January 29, 2025 20:22
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from 8330580 to d7b182d Compare January 29, 2025 20:39
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from db88046 to 1ff37b2 Compare January 29, 2025 20:40
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from d7b182d to d1cf72f Compare January 29, 2025 21:40
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from 1ff37b2 to 174743e Compare January 29, 2025 21:41
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from d1cf72f to cf11bbe Compare January 30, 2025 19:52
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from 174743e to 6399b8e Compare January 30, 2025 19:52
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from cf11bbe to aa8477e Compare January 31, 2025 19:12
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from 6399b8e to 467af85 Compare January 31, 2025 19:12
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from aa8477e to 490b5a6 Compare January 31, 2025 19:27
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from 467af85 to b7240de Compare January 31, 2025 19:27
@ehigham ehigham force-pushed the ehigham/ctx-coercer-cache branch from 490b5a6 to fcdedec Compare January 31, 2025 21:21
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from b7240de to c85bc8f Compare January 31, 2025 21:22
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.

1 participant