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

Separate the requirements file for each testing process #3297

Closed
wants to merge 1 commit into from

Conversation

Rebits
Copy link
Member

@Rebits Rebits commented Sep 15, 2022

Related issue
#3292

Description

Currently, every testing process makes use of a common requirement file. This is not scalable and produces several errors when more dependencies are added. I suggest changing this structure and splitting the requirements file into different requirements files.
Due to this being a breaking change and multiple CI processes depending on it, I suggest to maintain a general requirements file that will include all dependencies.

Added

  • Requirements directory
  • Requirements for several testing process: checkfiles, common, integration, performance, reliability, scan, system, tools

Updated

  • General requirements file
  • Add 1.26.5 for urllib3
  • Change requests version to >=2.23.0

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@user (Developer) ⚫⚫⚫ ⚫⚫⚫ Nothing to highlight
@user (Reviewer) ⚫⚫⚫ 🚫 🚫 🚫 Nothing to highlight

@Rebits
Copy link
Member Author

Rebits commented Sep 19, 2022

After a meeting with @jmv74211 we have agreed to research other alternatives to this issue. Also, we should create topics for:

  • Research dependency handlers
  • Research the use of currently required dependencies
  • Documentation of the use of the modules in the project

For now, this PR will be closed.

@Rebits Rebits closed this Sep 19, 2022
@Rebits Rebits mentioned this pull request Jul 19, 2023
@Rebits Rebits deleted the 3292-framework-error-w10 branch August 3, 2023 07:36
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.

1 participant