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

How to deal with readonly_files #211

Open
casparvl opened this issue Nov 28, 2024 · 0 comments
Open

How to deal with readonly_files #211

casparvl opened this issue Nov 28, 2024 · 0 comments

Comments

@casparvl
Copy link
Collaborator

  • We'll encourage the use of readonly_files in tests (e.g. in reviews), because most tests actually don't write to the files in their sourcedir
  • We can implement an option to quickly mark all files in the sourcesdir as readonly. E.g. test developer would set readonly_files = 'all', then the EESSI_Mixin class, in it's class body, would list all files in the sourcesdir and overwrite readonly_files with that list.
  • We'll initialize readonly_files = None in the EESSI_Mixin class body. Then, check after init if it's been set to anything else. If not, we print an error from EESSI_Mixin telling users that they have to set it explicitely (e.g. to [] if they don't want any readonly files).
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

No branches or pull requests

1 participant