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

Generate files as read-write. #1056

Merged
merged 1 commit into from
Sep 5, 2023
Merged

Generate files as read-write. #1056

merged 1 commit into from
Sep 5, 2023

Conversation

1uc
Copy link
Collaborator

@1uc 1uc commented Aug 31, 2023

Prior to this commit generated files were created read-only. When updating files, one would add write permissions and then overwrite the file. Even if it was read-only. This commit changes the behaviour to generate files with regular (read & write) permissions and doesn't change the permissions of files, e.g. it wont overwrite read-only files.

Prior to this commit generated files were created read-only. When
updating files, one would add write permissions and then overwrite the
file. Even if it was read-only. This commit changes the behaviour to
generate files with regular (read & write) permissions and doesn't
change the permissions of files, e.g. it wont overwrite read-only files.
@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (a1a035c) 69.98% compared to head (b698188) 69.98%.
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1056   +/-   ##
=======================================
  Coverage   69.98%   69.98%           
=======================================
  Files         188      188           
  Lines       25659    25659           
=======================================
  Hits        17957    17957           
  Misses       7702     7702           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bbpbuildbot
Copy link
Collaborator

Logfiles from GitLab pipeline #148649 (:white_check_mark:) have been uploaded here!

Status and direct links:

@1uc 1uc marked this pull request as ready for review August 31, 2023 16:52
@1uc
Copy link
Collaborator Author

1uc commented Aug 31, 2023

To check that the files are/should only generated in a build folder:
https://github.com/search?q=repo%3ABlueBrain%2Fnmodl%20code_generator.py&type=code

@1uc 1uc changed the title Generate files are read-write. Generate files as read-write. Sep 1, 2023
Copy link
Contributor

@pramodk pramodk left a comment

Choose a reason for hiding this comment

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

LGTM!

Ioannis and Nico also say we don't need to worry much about build folder files.

@pramodk pramodk merged commit 52ce1b3 into master Sep 5, 2023
16 checks passed
@pramodk pramodk deleted the 1uc/read-write branch September 5, 2023 14:33
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