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

Move tests to the test directory #4985

Merged
merged 4 commits into from
Dec 15, 2023
Merged

Move tests to the test directory #4985

merged 4 commits into from
Dec 15, 2023

Conversation

ferrarimarco
Copy link
Collaborator

@ferrarimarco ferrarimarco commented Dec 13, 2023 •
edited

Proposed Changes

  1. Move tests to the tests directory.
  2. Removed leftover TAPs.
  3. Removed unfilled test case README templates.
  4. Removed leftover docker test case. We need docker_hadolint only.
  5. Updated test cases README.

Readiness Checklist

Author/Contributor

  • I included all the needed documentation for this change.
  • I provided the necessary tests.

Reviewing Maintainer

  • Label as breaking if this is a large, fundamental change.
  • Label as either: automation, bug, documentation, enhancement, infrastructure.

BEGIN_COMMIT_OVERRIDE
chore: move tests to the test directory (#4985)
END_COMMIT_OVERRIDE

@ferrarimarco ferrarimarco added documentation Improvements or additions to documentation infrastructure related to project structure and maintenance labels Dec 13, 2023
@ferrarimarco ferrarimarco self-assigned this Dec 13, 2023
@ferrarimarco ferrarimarco marked this pull request as ready for review December 13, 2023 11:18
@ferrarimarco ferrarimarco added this pull request to the merge queue Dec 15, 2023
Merged via the queue into main with commit e6cf8d3 Dec 15, 2023
3 checks passed
@ferrarimarco ferrarimarco deleted the move-tests branch December 15, 2023 09:15
@github-actions github-actions bot mentioned this pull request Dec 21, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zkoppert zkoppert zkoppert approved these changes

@admiralAwkbar admiralAwkbar Awaiting requested review from admiralAwkbar

@lindluni lindluni Awaiting requested review from lindluni

@nemchik nemchik Awaiting requested review from nemchik

@Hanse00 Hanse00 Awaiting requested review from Hanse00 Hanse00 is a code owner

@GaboFDC GaboFDC Awaiting requested review from GaboFDC

Assignees

@ferrarimarco ferrarimarco

Labels
documentation Improvements or additions to documentation infrastructure related to project structure and maintenance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ferrarimarco @zkoppert