-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 DiscoverVariables hook implementation for test extension #8026
🌱 Add DiscoverVariables hook implementation for test extension #8026
Conversation
dc8e9a3
to
f9bcbff
Compare
d233eb4
to
e74137d
Compare
@killianmuldoon ping me when this one is ready for a pass |
Sure - there's a few PRs that probably need to be merged before this one is really worth looking at as it contains a bunch of those commits. |
e74137d
to
60103f0
Compare
a0f9191
to
47d9502
Compare
47d9502
to
22d2f98
Compare
22d2f98
to
0ba68e3
Compare
0ba68e3
to
51d8411
Compare
51d8411
to
f80a5e6
Compare
f80a5e6
to
b4406a4
Compare
b4406a4
to
5ad6f2f
Compare
5ad6f2f
to
572cb1a
Compare
572cb1a
to
ed70a92
Compare
Signed-off-by: killianmuldoon <kmuldoon@vmware.com>
ed70a92
to
717039a
Compare
/lgtm /assign @fabriziopandini |
LGTM label has been added. Git tree hash: 2a30c976af8673f8b7cb549aaf720ff8d1afeb89
|
/lgtm Let's get the basic test merged while we work on a more advanced test. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sbueringer The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Add an initial implementation of the DiscoverVariables hook to the test extension.
Part of #7985