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

fix: fix broken test after security URL change #3459

Closed
wants to merge 1 commit into from
Closed

fix: fix broken test after security URL change #3459

wants to merge 1 commit into from

Conversation

Avishagp
Copy link
Contributor

Fix broken test following security URL change

@Avishagp Avishagp requested review from a team as code owners July 14, 2022 11:44
@github-actions
Copy link
Contributor

github-actions bot commented Jul 14, 2022 •
edited

Messages
📖

You are modifying something in test/smoke directory, yet you are not on the branch starting with smoke/. You can prefix your branch with smoke/ and Smoke tests will trigger for this PR.

Generated by 🚫 dangerJS against 452c037

@maxjeffos
Copy link
Contributor

@Avishagp I'd suggest making your commit message a test: rather than a fix:

@Avishagp Avishagp closed this Jul 14, 2022
@Avishagp Avishagp deleted the fix/fix-broken-test branch July 14, 2022 12:00
This was referenced Nov 28, 2023
Open
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robcresswell robcresswell Awaiting requested review from robcresswell robcresswell was automatically assigned from snyk/snyk-open-source

@nirshirion nirshirion Awaiting requested review from nirshirion nirshirion was automatically assigned from snyk/snyk-open-source

@robh-snyk robh-snyk Awaiting requested review from robh-snyk robh-snyk was automatically assigned from snyk/snyk-open-source

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@Avishagp @maxjeffos