Cylc-flow: Remove executable bit of Python source code

Created on 25 Sep 2019  路  4Comments  路  Source: cylc/cylc-flow

Describe the bug

If you look at the Python source code under cylc, you will find some files that have the executable bit turned on. e.g. cylc.flow.scheduler_cli, cylc.flow.prerequisite, etc

image

I see no reason for having the files under cylc with +x.

Release version(s) and/or repository branch(es) affected?

master, 8.0a1

Steps to reproduce the bug

NA

Expected behavior

Source code simply with rw permissions.

Screenshots

Added in previous section.

Additional context

Pull requests welcome!
This is an Open Source project - please consider contributing a bug fix
yourself (please read CONTRIBUTING.md before starting any work though).

bug small

All 4 comments

I never submitted a PR like this, but I think it should work. Something like chmod -x $file..., then commit the file, and prepare the PR.

Doh!

(It probably results from running primitive built-in unit tests during development, before we had proper unit tests).

Closed by #3385.

Was this page helpful?
0 / 5 - 0 ratings