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

feat(conftest): Added root level conftest file #455

Conversation

hrushikesh430
Copy link
Collaborator

Added the conftest.py file at root level of repo, the fixture and configuratuions can be used all over repo.

Description

  • Added the conftest.py file at the root level. Added the config and build_dir fixture in the conftest.py.
  • Tested the file using in pre_encrypted_ota example pytest (PR).

@hrushikesh430 hrushikesh430 self-assigned this Dec 12, 2024
@hrushikesh430 hrushikesh430 force-pushed the feat/adding_root_level_conftest_file branch from 5fcc714 to 22e926e Compare December 13, 2024 06:04
@hrushikesh430
Copy link
Collaborator Author

@mahavirj @hfudev PTAL

Copy link
Member

@hfudev hfudev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@hrushikesh430 hrushikesh430 force-pushed the feat/adding_root_level_conftest_file branch from 22e926e to 265a596 Compare December 19, 2024 10:04
@mahavirj
Copy link
Member

@hrushikesh430 hrushikesh430 force-pushed the feat/adding_root_level_conftest_file branch from 265a596 to c2043bc Compare December 20, 2024 04:50
Added the conftest.py file at root level of repo, the fixture
and configuratuions can be used all over repo.
@hrushikesh430 hrushikesh430 force-pushed the feat/adding_root_level_conftest_file branch from c2043bc to f1c2cb1 Compare December 23, 2024 05:10
@hrushikesh430
Copy link
Collaborator Author

Right now, this PR is not needed. I will reopen after multiple CI configuration build is supported in this repository.

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.

3 participants