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

Configure Sweep #370

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Configure Sweep #370

wants to merge 16 commits into from

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Aug 7, 2023

🎉 Thank you for installing Sweep! We're thrilled to announce the latest update for Sweep, your trusty AI junior developer on GitHub. This PR creates a sweep.yaml config file, allowing you to personalize Sweep's performance according to your project requirements.

What's new?

  • Sweep is now configurable.
  • To configure Sweep, simply edit the sweep.yaml file in the root of your repository.
  • If you need help, check out the Sweep Default Config or Join Our Discord for help.

If you would like me to stop creating this PR, go to issues and say "Sweep: create an empty sweep.yaml file".
Thank you for using Sweep! 🧹

@sweep-ai sweep-ai bot added the sweep Assigns Sweep to an issue or pull request. label Aug 7, 2023
@sweep-ai
Copy link
Author

sweep-ai bot commented Aug 7, 2023

GitHub actions yielded the following error.

I'm sorry, but I cannot see the logs you mentioned. Could you please provide the logs so that I can assist you in extracting the relevant lines?

This is likely a linting or type-checking issue with the source code.

@sweep-ai
Copy link
Author

sweep-ai bot commented Aug 10, 2023

GitHub actions yielded the following error.

The command that failed is make: *** [Makefile:11: test] Error 1.

Here are the relevant lines from the logs:

FAIL	github.com/TykTechnologies/graphql-go-tools/pkg/subscription	0.633s
--- FAIL: TestTimeOutChecker (0.02s)
--- FAIL: TestTimeOutChecker/should_stop_process_if_timer_runs_out (0.02s)
time_out_test.go:52:
Error Trace:	/home/runner/work/graphql-go-tools/graphql-go-tools/pkg/subscription/time_out_test.go:52
Error:      	Should be true
Test:       	TestTimeOutChecker/should_stop_process_if_timer_runs_out
FAIL

Please note that the specific error message is Should be true in the time_out_test.go file.

There are a lot of errors. This is likely due to a small parsing issue or a missing import with the files changed in the PR.

This is likely a linting or type-checking issue with the source code. Update the code the changes and avoid modifying the existing tests.

@sweep-ai
Copy link
Author

sweep-ai bot commented Aug 17, 2023

GitHub actions yielded the following error.

The command that failed is make ci. Here are the relevant lines from the logs:

--- FAIL: TestOpenAPI_v3_0_0 (0.02s)
--- FAIL: TestOpenAPI_v3_0_0/example_oas7.json (0.00s)
openapi_test.go:30:
Error Trace:    /home/runner/work/graphql-go-tools/graphql-go-tools/pkg/openapi/openapi_test.go:30
/home/runner/work/graphql-go-tools/graphql-go-tools/pkg/openapi/openapi_test.go:44
Error:          Not equal:
expected: "schema {\n    query: Query\n    mutation: Mutation\n}\n\ntype Query {\n    \"Find a device by name.\"\n    findDeviceByName(deviceName: String!): Device\n    \"Return a device collection.\"\n    findDevices: [Device]\n    \"Return a user.\"\n    user: User\n}\n\ntype Mutation {\n    \"Create and return a device.\"\n    createDevice(deviceInput: DeviceInput!): Device\n    \"Replace a device by name.\"\n    replaceDeviceByName(deviceInput: DeviceInput!, deviceName: String!): Device\n}\n\ntype Device {\n    \"The device name in the network\"\n    name: String!\n    status: Boolean\n    \"The device owner Name\"\n    userName: String!\n}\n\ninput DeviceInput {\n    name: String!\n    status: Boolean\n    userName: String!\n}\n\n\"A user represents a natural person\"\ntype User {\n    \"The legal name of a user\"\n    name: String\n}"
actual  : "schema {\n    query: Query\n    mutation: Mutation\n}\n\ntype Query {\n    \"Find a device by name.\"\n    findDeviceByName(deviceName: String!): Device\n    \"Return a device collection.\"\n    findDevices: [Device]\n    \"Return a user.\"\n    user: User\n}\n\ntype Mutation {\n    \"Create and return a device.\"\n    createDevice(deviceInput: DeviceInput!): Device\n    \"Replace a device by name.\"\n    replaceDeviceByName(deviceInput: DeviceInput!, deviceName: String!): Device\n}\n\n\"A device is an object connected to the network\"\ntype Device {\n    \"The device name in the network\"\n    name: String!\n    status: Boolean\n    \"The device owner Name\"\n    userName: String!\n}\n\ninput DeviceInput {\n    name: String!\n    status: Boolean\n    userName: String!\n}\n\n\"A user represents a natural person\"\ntype User {\n    \"The legal name of a user\"\n    name: String\n}"

Diff:
--- Expected
+++ Actual
@@ -21,2 +21,3 @@

+"A device is an object connected to the network"
type Device {
Test:        TestOpenAPI_v3_0_0/example_oas7.json
FAIL
FAIL    github.com/TykTechnologies/graphql-go-tools/pkg/openapi    0.025s

Please note that the relevant lines may vary depending on the specific error you are debugging.

There are a lot of errors. This is likely due to a small parsing issue or a missing import with the files changed in the PR.

This is likely a linting or type-checking issue with the source code. Update the code changed by the PR. Don't modify the existing tests.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants