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

enable spotbugs for db-sources submodule #36705

Conversation

stephane-airbyte
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Mar 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Apr 3, 2024 9:49pm

@octavia-squidington-iii octavia-squidington-iii added the CDK Connector Development Kit label Mar 29, 2024
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 1c4ccac to 8c92b70 Compare April 1, 2024 15:19
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch from d66224f to c2f86c1 Compare April 1, 2024 15:19
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 8c92b70 to 8315c7e Compare April 1, 2024 19:37
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch from c2f86c1 to c08f788 Compare April 1, 2024 19:37
@@ -155,7 +155,7 @@ class PythonSourceAcceptanceTest : SourceAcceptanceTest() {
private val LOGGER: Logger = LoggerFactory.getLogger(PythonSourceAcceptanceTest::class.java)
private const val OUTPUT_FILENAME = "output.json"

lateinit var IMAGE_NAME: String
var IMAGE_NAME: String = "dummy_image_name"
Copy link
Contributor

Choose a reason for hiding this comment

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

I was going to post a comment about how this is bad but then I noticed that this class was PythonSourceAcceptanceTest which assuredly is dead code.

@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 8315c7e to 34b730f Compare April 2, 2024 14:55
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch from c08f788 to 185fc3b Compare April 2, 2024 14:55
@stephane-airbyte stephane-airbyte marked this pull request as ready for review April 2, 2024 14:56
@stephane-airbyte stephane-airbyte requested a review from a team as a code owner April 2, 2024 14:56
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 34b730f to 8d4840e Compare April 2, 2024 18:07
@stephane-airbyte stephane-airbyte requested a review from a team as a code owner April 2, 2024 18:07
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 8d4840e to 8b6a839 Compare April 2, 2024 18:09
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch from 185fc3b to af00bdf Compare April 2, 2024 18:09
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 8b6a839 to 796089c Compare April 3, 2024 16:29
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch from af00bdf to 383be2e Compare April 3, 2024 16:29
Copy link
Contributor Author

stephane-airbyte commented Apr 3, 2024

Merge activity

@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 796089c to 910e05c Compare April 3, 2024 17:57
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch from 383be2e to 3130d1c Compare April 3, 2024 17:57
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule branch from 910e05c to 8d0610a Compare April 3, 2024 21:40
Base automatically changed from stephane/03-28-enabling_postbugs_for_datastore-postgres_submodule to master April 3, 2024 21:48
@stephane-airbyte stephane-airbyte force-pushed the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch from 3130d1c to ca16f89 Compare April 3, 2024 21:49
@stephane-airbyte stephane-airbyte merged commit fad0737 into master Apr 3, 2024
27 checks passed
@stephane-airbyte stephane-airbyte deleted the stephane/03-28-enable_spotbugs_for_db-sources_submodule branch April 3, 2024 22:28
nurikk pushed a commit to nurikk/airbyte that referenced this pull request Apr 4, 2024
markcusack pushed a commit to markcusack/airbyte that referenced this pull request Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CDK Connector Development Kit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants