Description

Introduces ability to trigger workflows when an issue is found by a check.

Summary generated by Reviewpad on 11 Aug 23 18:47 UTC

This pull request includes the following changes:

1. In the mocks.go file, a new nil element is added to a function call. This change does not seem to have a significant impact but may be necessary for the overall functionality of the code.

2. The run.go file has been modified. On line 152, an additional argument nil is added to a function call to the reviewpad.Run function. These changes appear to be related to adjusting the parameters passed to the reviewpad.Run function.

3. Multiple function calls in different test cases have been modified in the mocks_test.go file. Specifically, a nil parameter is added in the functions TestNewEvalEnv_WhenGetPullRequestFilesFails, TestNewEvalEnv_WhenNewFileFails, TestNewEvalEnv, and TestNewEvalEnv_WhenGetPullRequestFails.

4. The exec.go file contains changes to the ExecConfigurationFile function. The changes introduce a check to determine if a workflow should run based on a triggering check. The function checks if any of the checks in checksWithIssues match the triggerOnCheck field of the workflow. If a match is found, the workflow should run; otherwise, it continues to the next workflow.

5. The env.go file has a new method GetChecksWithIssues() added to the Interpreter interface.

6. The loader.go file has changes in the transform function. The changes include adding a new field TriggerOnCheck to the transformedWorkflows struct in the append function call.

7. In the integration_test.go file, on line 330, a new argument nil is added to the function reviewpad.Run().

8. The inline_rules_normalizer.go file contains changes to the processWorkflow function. These changes include adding the field TriggerOnCheck to the wf struct initialization.

9. The interpreter_internal_test.go file has changes related to adding nil values to the arguments of certain function calls in the test cases TestNewInterpreter_WhenNewEvalEnvFails, TestNewInterpreter, and TestReportMetric.

10. The interpreter.go file introduces a new function GetChecksWithIssues() to retrieve a slice of strings. Additionally, the NewInterpreter function is modified to accept an additional parameter checksWithIssues of type []string.

11. In the engine/lang.go file, a new field TriggerOnCheck is added to the PadWorkflow struct.

12. The engine/exec_test.go file includes two changes. On line 444 and line 719, new nil values are added.

13. The lang/type_internal_test.go file has test cases added for the Equals() function. These tests compare various types and assert that the Equals() function correctly identifies identical types.

14. Changes in the runner.go file include a new parameter checksWithIssue added to the Run function. This parameter is used in the call to aladino.NewInterpreter. The aladino.NewInterpreter call is updated to include the checksWithIssue parameter.

15. The env.go file introduces changes to the Env interface and the BaseEnv struct. The Env interface now includes a new method GetChecksWithIssues(), which returns a slice of strings. The BaseEnv struct includes a new field ChecksWithIssues of type []string. The GetChecksWithIssues() method is implemented in the BaseEnv struct to return the ChecksWithIssues field. The NewEvalEnv() function is also updated to receive a new parameter checksWithIssues of type []string and assign it to the ChecksWithIssues field of the BaseEnv struct.

Please review these changes and ensure they align with the requirements of the system.

🤖 Generated by Copilot at 05ba2b6

Add TriggerOnCheck field to PadWorkflow type to enable custom workflows based on checks. This change affects the engine/lang.go file.

Code review and merge strategy

Ship: this pull request can be automatically merged and does not require code review

<! -- Ask: this pull request requires a code review before merge -->

How

🤖 Generated by Copilot at 05ba2b6

• Add a new field TriggerOnCheck to the PadWorkflow type to specify a check name that triggers the workflow execution (link)