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

Match the smoke test invocation of CLI/proxy #7576

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

jeffwidman
Copy link
Member

@jeffwidman jeffwidman commented Jul 17, 2023

This updates the command to match the smoke invocation used in the CLI and proxy.

This should hopefully reduce mistakes creeping in like dependabot/cli#148 where I copy/pasted not realizing there was a slight difference between the two repos.

Related:

jeffwidman added a commit to dependabot/cli that referenced this pull request Jul 17, 2023
This particular command is getting long, so break it across multiple lines.

This will also make the command invocation more consistent across the smoke workflows for core/proxy/cli repos.

Related:
* dependabot/dependabot-core#7576
* #148
This updates the command to match the smoke invocation used in the CLI and proxy.

This should hopefully reduce mistakes creeping in like dependabot/cli#148 where I copy/pasted not realizing there was a slight difference between the two repos.
@jeffwidman jeffwidman force-pushed the match-cli-proxy-smoke-invocation branch from 47e2a0d to 1e08fc4 Compare July 17, 2023 19:57
@jeffwidman jeffwidman enabled auto-merge (squash) July 17, 2023 19:57
@jeffwidman jeffwidman merged commit 2e5cd7c into main Jul 17, 2023
86 checks passed
@jeffwidman jeffwidman deleted the match-cli-proxy-smoke-invocation branch July 17, 2023 20:01
brettfo pushed a commit to brettfo/dependabot-core that referenced this pull request Oct 11, 2023
This updates the command to match the smoke invocation used in the CLI and proxy.

This should hopefully reduce mistakes creeping in like dependabot/cli#148 where I copy/pasted not realizing there was a slight difference between the two repos.

Related:
* dependabot/cli#150
* dependabot/cli#148
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants