Modify run_validation.py
Run_validation rejects any changes made in a merge request to files outside the primitives folder in this repository, including "archived" and "failed" primitives. This suggests that no contributor is allowed to make changes to any of these other folders/files other than an owner of the repository who directly modifies the master branch themselves. I believe @mitar used to make most of these non-primitive changes directly to the master branch.
Due to the current structure of the project and in the interest of maintaining a record of all changes, I suggest all changes to the primitives branch to go through merge requests, including changes to files/folders outside of the primitives folder. Therefore, I recommend these 2 lines be removed so that merge requests attempting to modify non-primitive files can still undergo CI checks without error:
if any(not (file_path == '.gitattributes' or file_path.startswith('primitives/')) for file_path in files_changed):
raise ValidationError("There are non-primitive file changes between your branch and upstream. If you have not authored those changes, you need to update your branch with upstream.")
https://gitlab.com/datadrivendiscovery/primitives/-/blob/master/run_validation.py#L907
This issue is currently why my MR to move a few primitives outside of the "failed" folder will not even run validation. https://gitlab.com/bhshah/primitives/-/jobs/1718107020