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

docs(test_utils): Update postman docs #3055

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Conversation

pixincreate
Copy link
Member

Type of Change

  • Refactoring
  • Documentation

Description

Update postman documentation since we had previously made some changes w.r.t how the postman commands are passed.

Motivation and Context

I don't want to make people scream if it does not work.

How did you test it?

This command should fail to run postman tests

cargo run --package test_utils --bin test_utils -- --connector_name=<connector_name> --base_url=<base_url> --admin_api_key=<admin_api_key>

This command should run postman tests as expected.

cargo run --package test_utils --bin test_utils -- --connector-name=<connector_name> --base-url=<base_url> --admin-api-key=<admin_api_key>

Checklist

  • I reviewed the submitted code

@pixincreate pixincreate added C-doc Category: Documentation updates S-waiting-on-approval Status: Review completed, awaiting for approvals from code-owners labels Dec 5, 2023
@pixincreate pixincreate added this to the November 2023 Release milestone Dec 5, 2023
@pixincreate pixincreate self-assigned this Dec 5, 2023
@pixincreate pixincreate requested a review from a team as a code owner December 5, 2023 07:33
@pixincreate pixincreate enabled auto-merge December 5, 2023 07:37
@pixincreate pixincreate added this pull request to the merge queue Dec 5, 2023
Merged via the queue into main with commit 8b7a7aa Dec 5, 2023
12 of 14 checks passed
@pixincreate pixincreate deleted the update-postman-doc branch December 5, 2023 08:27
@pixincreate pixincreate removed the S-waiting-on-approval Status: Review completed, awaiting for approvals from code-owners label Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-doc Category: Documentation updates
Projects
No open projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants