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

[WIZ] - Remove unused test_requirements #3261

Merged
merged 1 commit into from
Jan 13, 2025
Merged

[WIZ] - Remove unused test_requirements #3261

merged 1 commit into from
Jan 13, 2025

Conversation

flavienSindou
Copy link
Contributor

Proposed changes

  • Remove unused test-requirement

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality using different use cases (Run tests locally)
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

N.A.

@flavienSindou flavienSindou self-assigned this Jan 13, 2025
@flavienSindou flavienSindou added bug use for describing something not working as expected filigran team use to identify PR from the Filigran team critical use to identify critical bug to fix ASAP test automation use to identify issue related to test automation implementation and removed test automation use to identify issue related to test automation implementation labels Jan 13, 2025
@flavienSindou flavienSindou linked an issue Jan 13, 2025 that may be closed by this pull request
[WIZ] - Connector code breaks the CI/CD because it uses test_requirements but has no tests #3260
Closed
@flavienSindou flavienSindou merged commit dd13cd2 into master Jan 13, 2025
4 checks passed
@flavienSindou flavienSindou deleted the bug/3260-wiz--connector-code-breaks-the-cicd-because-it-uses-test_requirements-but-has-no-tests branch January 13, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Powlinett Powlinett Powlinett approved these changes

@Megafredo Megafredo Awaiting requested review from Megafredo

@helene-nguyen helene-nguyen Awaiting requested review from helene-nguyen

Assignees

@flavienSindou flavienSindou

Labels
bug use for describing something not working as expected critical use to identify critical bug to fix ASAP filigran team use to identify PR from the Filigran team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[WIZ] - Connector code breaks the CI/CD because it uses test_requirements but has no tests
2 participants
@flavienSindou @Powlinett