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

[TT-500][TT-682]Add build and lint to integration-tests #11221

Merged
merged 27 commits into from
Nov 13, 2023

Conversation

tateexon
Copy link
Contributor

@tateexon tateexon commented Nov 7, 2023

covers TT-500 and TT-682

Copy link
Contributor

github-actions bot commented Nov 7, 2023

I see that you haven't updated any README files. Would it make sense to do so?

@tateexon tateexon force-pushed the TT-653-Migrate-CTF-Packages-To-CTF branch from 1994038 to 50b2d40 Compare November 7, 2023 22:31
integration-tests/contracts/contract_vrf_models.go Outdated Show resolved Hide resolved
integration-tests/universal/log_poller/helpers.go Outdated Show resolved Hide resolved
integration-tests/universal/log_poller/helpers.go Outdated Show resolved Hide resolved
integration-tests/docker/test_env/cl_node.go Outdated Show resolved Hide resolved
@tateexon tateexon marked this pull request as ready for review November 7, 2023 22:44
@tateexon tateexon requested review from a team as code owners November 7, 2023 22:44
@@ -18,35 +17,35 @@ import (
"github.com/ethereum/go-ethereum/common"
geth_types "github.com/ethereum/go-ethereum/core/types"
"github.com/rs/zerolog"
"github.com/smartcontractkit/wasp"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

btw. maybe that whole folder's content should be located in integration-tests/actions to make it more consistent with existing structure?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of the scope of this PR but something we can definitely look into doing.

@tateexon tateexon requested a review from a team November 8, 2023 22:03
@tateexon tateexon changed the title Add build and lint to integration-tests [TT-500][TT-682]Add build and lint to integration-tests Nov 9, 2023
@tateexon tateexon enabled auto-merge November 9, 2023 17:48
@cl-sonarqube-production
Copy link

SonarQube Quality Gate

Quality Gate passed

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

No Coverage information No Coverage information
1.7% 1.7% Duplication

@tateexon tateexon added this pull request to the merge queue Nov 13, 2023
Merged via the queue into develop with commit b66b723 Nov 13, 2023
87 checks passed
@tateexon tateexon deleted the TT-653-Migrate-CTF-Packages-To-CTF branch November 13, 2023 16:21
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.

6 participants