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: fix toml format to address wasm build failure #6967

Merged
merged 18 commits into from
Jan 28, 2025

Conversation

JeevaRamu0104
Copy link
Contributor

@JeevaRamu0104 JeevaRamu0104 commented Jan 2, 2025

Type of Change

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

Description

An incorrect TOML format has been added to the ctp_mastercard due to which the wasm build is getting failed.

Add development as feature in wasm config

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?

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

@JeevaRamu0104 JeevaRamu0104 requested review from a team as code owners January 2, 2025 09:21
Copy link

Review changes with  SemanticDiff

@JeevaRamu0104 JeevaRamu0104 self-assigned this Jan 2, 2025
@JeevaRamu0104 JeevaRamu0104 added the A-euclid-foc Area: Euclid Family of Crates label Jan 2, 2025
@JeevaRamu0104 JeevaRamu0104 linked an issue Jan 6, 2025 that may be closed by this pull request
@pixincreate pixincreate changed the title chore: wasm jenkins build chore: fix toml format to address wasm build failure Jan 6, 2025
swangi-kumari
swangi-kumari previously approved these changes Jan 6, 2025
SanchithHegde
SanchithHegde previously approved these changes Jan 7, 2025
Copy link

codecov bot commented Jan 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@7fd3551). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6967   +/-   ##
=======================================
  Coverage        ?    4.12%           
=======================================
  Files           ?     1010           
  Lines           ?   207403           
  Branches        ?        0           
=======================================
  Hits            ?     8548           
  Misses          ?   198855           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jan 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jan 24, 2025
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jan 27, 2025
@Gnanasundari24 Gnanasundari24 removed this pull request from the merge queue due to a manual request Jan 27, 2025
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jan 28, 2025
Merged via the queue into main with commit ecab2b1 Jan 28, 2025
18 of 21 checks passed
@Gnanasundari24 Gnanasundari24 deleted the fix-wasm-jenkins-build branch January 28, 2025 09:17
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-euclid-foc Area: Euclid Family of Crates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CHORE]: wasm build fix
5 participants