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.

Sign up for GitHub

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

Jump to bottom

feat: Feature/cli add parameters to specify the bruno collection path instead of cwd #2968

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

feat: Feature/cli add parameters to specify the bruno collection path instead of cwd #2968

wants to merge 8 commits into from

Conversation

berlingoqc
Copy link

@berlingoqc berlingoqc commented Aug 29, 2024 •
edited
Loading

Description

Add the possibility to run the cli outside of the Bruno collection folder.

Contribution Checklist:

  • The pull request only addresses one issue or adds one feature.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or gifs to help explain the change if applicable.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.
      --collection    Path to the folder where the bruno collection is
                           stored, default to current working directory
                                                          [string] [default: ""]

➜  bru run AGENT--env nonprod --collection ../../../my-collections/
Running Folder 

AGENT/Create Agent (401 ) - 576 ms
   ✓ ressource is ok
AGENT/Get Agent (401 ) - 284 ms
   ✓ ressource is ok
AGENT/Delete Agent (401 ) - 232 ms
   ✓ ressource is deleted

Requests:    3 passed, 3 total
Tests:       3 passed, 3 total
Assertions:  0 passed, 0 total

Resolves #2967

@berlingoqc berlingoqc mentioned this pull request Aug 29, 2024
Open
1 task
@pull-request-size pull-request-size bot added size/L and removed size/S labels Aug 30, 2024
@berlingoqc berlingoqc mentioned this pull request Sep 5, 2024
Open
5 tasks
Pragadesh-45 and others added 6 commits September 5, 2024 11:44
@Pragadesh-45 @berlingoqc
style chore: make delete div bg important (usebruno#2949)
dac23a2
@ParamjotSingh5 @helloanoop @berlingoqc
feat(usebruno#736): Switch tabs with keyboard shortcut (usebruno#812)
abfd672 
* feat(usebruno#736): Switch tabs with keyboard shortcut

1. Registered keyboard events in Hotkeys/index.js
2. Added logic for replacing `state.activeTabUid` to switch active tab as per keyboard event.
3. Maintained a stack `recentUsedTabsStack` for tab visit history and pop out on `Ctrl+Tab` keyboard event.

* feat(usebruno#736): Switch tabs with keyboard shortcut

Keeping this feature request only limited to CTRL+PGUP and CTRL_PGDN button clicks functionality. Hence removing logic for CTRL+TAB click functionality.

* feat(usebruno#736): Switch tabs with keyboard shortcut

clean up

* feate(usebruno#827): Switch tabs with keyboard shortcut

* Implimented logic of cyclic traversal of tabs array with % opreator.

---------

Co-authored-by: Anoop M D <[email protected]>
@pietrygamat @helloanoop @berlingoqc
fix: Issue with Parameters Passed in the URL(usebruno#2124) (usebruno…
d0ad06e 
…#2139)

* fix: Issue with Parameters Passed in the URL(usebruno#2124)

The '=' should be allowed within query parameter value. While first equals sign separates parameter name from its value, any subsequent occurrences are valid and should not be discarded.
The '#' in URL always indicates the start of URI Fragment component, and should not be treated as part of any parameter value.

* chore: gracefully fail when URLSearchParams throws error

---------

Co-authored-by: Anoop M D <[email protected]>
@lohxt1 @berlingoqc
fix: option to toggle on/off system proxy env variables (usebruno#2724)
a1d8400 
fix: option to toggle on/off system proxy env variables
@helloanoop @berlingoqc
feat: update proxy implementation in preferences (usebruno#2977)
b7a298c
@berlingoqc
add collection path
dc7b4f4
@berlingoqc berlingoqc force-pushed the feature/cli-specific-collection-path branch from 2e60d5f to dc7b4f4 Compare September 5, 2024 15:46
@pull-request-size pull-request-size bot added size/XL and removed size/L labels Sep 5, 2024
@pull-request-size pull-request-size bot added size/S and removed size/XL labels Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CLI] add parameters to specify the bruno collection path instead of cwd
6 participants
@berlingoqc @Pragadesh-45 @ParamjotSingh5 @pietrygamat @lohxt1 @helloanoop