Skip to content

Conversation

@gopikrishna000
Copy link
Contributor

@gopikrishna000 gopikrishna000 commented Dec 10, 2025

Description

enable clippy for connector integration, etc crates.
and fix existing warnings

Motivation and Context

To prevent panic based codes in production.
Notice the issue at PR checks.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies application configuration/environment variables

How did you test it?

@gopikrishna000 gopikrishna000 requested review from a team as code owners December 10, 2025 06:37
@gopikrishna000 gopikrishna000 force-pushed the enable-clippy-connector-crate branch from 54d3344 to ac1fddc Compare December 10, 2025 17:39
@gopikrishna000 gopikrishna000 changed the title enable clippy for connector integration crate feat: Enable clippy for connector integration crate Dec 10, 2025
@gopikrishna000 gopikrishna000 force-pushed the enable-clippy-connector-crate branch 2 times, most recently from ed75ffa to d184bf4 Compare December 10, 2025 19:00
Shubhodip900
Shubhodip900 previously approved these changes Dec 11, 2025
Saptak88
Saptak88 previously approved these changes Dec 11, 2025
@bernard-eugine bernard-eugine added this pull request to the merge queue Dec 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 11, 2025
@gopikrishna000 gopikrishna000 force-pushed the enable-clippy-connector-crate branch 5 times, most recently from 86899ef to d29d1e9 Compare December 12, 2025 04:23
@AmitsinghTanwar007 AmitsinghTanwar007 added this pull request to the merge queue Dec 12, 2025
Merged via the queue into main with commit a03dfc2 Dec 12, 2025
14 of 15 checks passed
@AmitsinghTanwar007 AmitsinghTanwar007 deleted the enable-clippy-connector-crate branch December 12, 2025 07:26
pixincreate added a commit that referenced this pull request Dec 12, 2025
…pmorgan

* 'main' of github.com:juspay/connector-service:
  feat: Enable clippy for connector integration crate (#359)
  chore(version): 2025.12.12.0
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.

5 participants