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: create justfile for running commands for v1 and v2 migrations #5325

Merged
merged 21 commits into from
Jul 16, 2024

Conversation

hrithikesh026
Copy link
Contributor

@hrithikesh026 hrithikesh026 commented Jul 15, 2024

Type of Change

  • Enhancement

Description

created justfile for running commands for v1 and v2 migrations. This PR also updates the CI to include the new commands for running migrations.

https://github.com/casey/just

Get started by installing just package.
cargo install just

then run just --list to get a list of supported just commands.

To run v1 migrations run:
just migrate

To run v2 migrations run:
just migrate_v2

Motivation and Context

Using Just provides an easier way to setup commands that would have been a little difficult using the makefile.

How did you test it?

Ran just --list

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code

@hrithikesh026 hrithikesh026 added the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Jul 15, 2024
@hrithikesh026 hrithikesh026 added this to the July 2024 Release milestone Jul 15, 2024
@hrithikesh026 hrithikesh026 self-assigned this Jul 15, 2024
@hrithikesh026 hrithikesh026 requested a review from a team as a code owner July 15, 2024 11:38
@Narayanbhat166 Narayanbhat166 requested review from a team as code owners July 15, 2024 15:53
@hyperswitch-bot hyperswitch-bot bot added the M-database-changes Metadata: This PR involves database schema changes label Jul 15, 2024
@hyperswitch-bot hyperswitch-bot bot removed the M-database-changes Metadata: This PR involves database schema changes label Jul 15, 2024
@hyperswitch-bot hyperswitch-bot bot added the M-database-changes Metadata: This PR involves database schema changes label Jul 15, 2024
Narayanbhat166
Narayanbhat166 previously approved these changes Jul 15, 2024
justfile Outdated Show resolved Hide resolved
justfile Outdated Show resolved Hide resolved
justfile Outdated Show resolved Hide resolved
justfile Outdated Show resolved Hide resolved
justfile Outdated Show resolved Hide resolved
justfile Outdated Show resolved Hide resolved
.github/workflows/postman-collection-runner.yml Outdated Show resolved Hide resolved
.github/workflows/migration-check.yaml Outdated Show resolved Hide resolved
.github/workflows/migration-check.yaml Outdated Show resolved Hide resolved
.github/workflows/migration-check.yaml Show resolved Hide resolved
@Narayanbhat166 Narayanbhat166 requested a review from a team as a code owner July 16, 2024 06:51
jarnura
jarnura previously approved these changes Jul 16, 2024
Copy link
Member

@jarnura jarnura left a comment

Choose a reason for hiding this comment

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

LGTM

Narayanbhat166
Narayanbhat166 previously approved these changes Jul 16, 2024
SanchithHegde
SanchithHegde previously approved these changes Jul 16, 2024
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jul 16, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 16, 2024
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jul 16, 2024
Merged via the queue into main with commit 23bfceb Jul 16, 2024
11 of 13 checks passed
@Gnanasundari24 Gnanasundari24 deleted the create-justfile-for-running-v2-migrations branch July 16, 2024 12:21
@pixincreate pixincreate removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Jul 16, 2024
pixincreate added a commit that referenced this pull request Jul 16, 2024
* 'main' of github.com:juspay/hyperswitch: (25 commits)
  fix(logs): ignore request headers while logging (#5273)
  feat(webhooks): add support for custom outgoing webhook http headers (#5275)
  fix(payment_methods): set `requires_cvv` to false when either `connector_mandate_details` or `network_transaction_id` is present during MITs (#5331)
  chore: create justfile for running commands for v1 and v2 migrations (#5325)
  fix(routing): do not update `perform_session_flow_routing` output if the `SessionRoutingChoice` is none (#5336)
  fix(database): modified_at updated for every state change for Payment Attempts (#5312)
  feat(mca): Added recipient connector call for open banking connectors (#3758)
  chore(version): 2024.07.16.0
  refactor(connector): [Mifinity] add a field language_preference in payment request for mifinity payment method data (#5326)
  fix(router): store `customer_acceptance` in payment_attempt, use it in confirm flow for delayed authorizations like external 3ds flow (#5308)
  feat(proxy): add support to pass proxy bypass urls from configs (#5322)
  Docs: Updating Error codes in API-ref (#5296)
  feat(core): [Payouts] Add retrieve flow for payouts (#4936)
  fix(connector): [AUTHORIZEDOTNET] Populate error reason for failure transactions (#5319)
  chore(version): 2024.07.15.0
  feat(logging): Emit a setup error when a restricted keys are used for logging default keys (#5185)
  feat(payment_methods): add support to migrate existing customer PMs from processor to hyperswitch (#5306)
  feat(connector): [DATATRANS] Implement card payments (#5028)
  chore: making of function create_encrypted_data (#5251)
  fix(payments): populate merchant order ref id in list (#5310)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M-database-changes Metadata: This PR involves database schema changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants