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

bump ctf #498

Merged
merged 1 commit into from
Aug 1, 2024
Merged

bump ctf #498

merged 1 commit into from
Aug 1, 2024

Conversation

jmank88
Copy link
Contributor

@jmank88 jmank88 commented Aug 1, 2024

No description provided.

@@ -79,7 +79,7 @@ require (
github.com/cespare/xxhash v1.1.0 // indirect
github.com/cespare/xxhash/v2 v2.2.0 // indirect
github.com/chai2010/gettext-go v1.0.2 // indirect
github.com/chaos-mesh/chaos-mesh/api/v1alpha1 v0.0.0-20220226050744-799408773657 // indirect
github.com/chaos-mesh/chaos-mesh/api v0.0.0-20240709130330-9f4feec7553f // indirect
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had to manually align this one with CTF to get the mod resolution to work.

Copy link
Collaborator

@aalu1418 aalu1418 Aug 1, 2024

Choose a reason for hiding this comment

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

just a guess, do you need to add this line? I had to add a bunch of things to the integration test go.mod a while back for solana

https://github.com/smartcontractkit/chainlink/blob/3b4c2b58c3ebb04a2261108e758a3419de436a71/integration-tests/go.mod#L488

exclude github.com/chaos-mesh/chaos-mesh/api/v1alpha1 v0.0.0-20220226050744-799408773657

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That would have helped since I replaced that version with this one. I think it is only a problem because the tooling assumes that import paths won't step on each other, so each package only belong to one module. Should be fine now that we are past it.

@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@jmank88 jmank88 merged commit b5d9082 into develop Aug 1, 2024
21 checks passed
@jmank88 jmank88 deleted the ctf-bump branch August 1, 2024 17:44
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.

2 participants