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

chore: run clippy with default number of jobs in github workflows #7088

Merged
merged 4 commits into from
Jan 29, 2025

Conversation

hrithikesh026
Copy link
Contributor

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

changed cargo clippy command in workflow files to use default number of jobs instead of 2.

Additional Changes

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

Motivation and Context

How did you test it?

Github CI checks has passed.

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@hrithikesh026 hrithikesh026 self-assigned this Jan 22, 2025
@hrithikesh026 hrithikesh026 requested a review from a team as a code owner January 22, 2025 09:20
Copy link

Review changes with  SemanticDiff

SanchithHegde
SanchithHegde previously approved these changes Jan 22, 2025
kashif-m
kashif-m previously approved these changes Jan 22, 2025
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jan 27, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jan 27, 2025
@hrithikesh026 hrithikesh026 dismissed stale reviews from kashif-m and SanchithHegde via 9121252 January 28, 2025 08:20
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jan 29, 2025
Merged via the queue into main with commit 337095b Jan 29, 2025
18 of 21 checks passed
@Gnanasundari24 Gnanasundari24 deleted the remove-jobs-arguement-from-clippy branch January 29, 2025 09:06
@SanchithHegde SanchithHegde added the A-CI-CD Area: Continuous Integration/Deployment label Jan 29, 2025
pixincreate added a commit that referenced this pull request Jan 30, 2025
…ress

* 'main' of github.com:juspay/hyperswitch:
  chore(version): 2025.01.30.0
  Documentation edits made through Mintlify web editor
  feat(connector): add template code for chargebee (#7036)
  chore: run clippy with default number of jobs in github workflows (#7088)
  feat(router): add accept-language from request headers into browser-info (#7074)
  refactor(euclid): update proto file for elimination routing (#7032)
  chore(version): 2025.01.29.0
  refactor(router): prioritise `connector_mandate_id` over `network_transaction_id` during MITs (#7081)
  chore: add stripe to network transaction id support (#7096)
  fix(multitenancy): add a fallback for get commands in redis (#7043)
  refactor(currency_conversion): re frame the currency_conversion crate to make api calls on background thread (#6906)
  chore: fix `toml` format to address wasm build failure (#6967)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI-CD Area: Continuous Integration/Deployment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants