Skip to content

Additional GCP data fetchers#119

Open
mjsprengers wants to merge 31 commits intoBlazebit:mainfrom
mjsprengers:additional_gcp_fetchers
Open

Additional GCP data fetchers#119
mjsprengers wants to merge 31 commits intoBlazebit:mainfrom
mjsprengers:additional_gcp_fetchers

Conversation

@mjsprengers
Copy link
Contributor

No description provided.

mjsprengers and others added 30 commits October 30, 2024 09:21
/**
* Fetches VPC Firewall rules for GCP projects.
*
* @author Blazebit
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you check @author/@since

catch (PermissionDeniedException e) {
if ( e.getCause() instanceof HttpResponseException
&& ((HttpResponseException) e.getCause()).getContent()
.contains( "\"accessNotConfigured\"" ) ) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

What does this mean? Shouldn't we somehow surface this?


junit-jupiter = { module = "org.junit.jupiter:junit-jupiter", version.ref = "junit-jupiter" }
junit-platform-launcher = { module = "org.junit.platform:junit-platform-launcher" }
junit-platform-launcher = { module = "org.junit.platform:junit-platform-launcher", version = "1.11.3" }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why this?

Copy link
Collaborator

Choose a reason for hiding this comment

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

When do we update these dependencies?

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