Skip to content

Added option not to compile python files - #133

Open
askvortsov1 wants to merge 6 commits into
jrief:masterfrom
askvortsov1:master
Open

Added option not to compile python files#133
askvortsov1 wants to merge 6 commits into
jrief:masterfrom
askvortsov1:master

Conversation

@askvortsov1

Copy link
Copy Markdown

No description provided.

@AndreasBackx AndreasBackx left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM with the changes I mentioned, but gonna leave the final review to @jrief. Thank you for your PR.

PS: In the future please use a different branch name for your PR. See https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests.

Comment thread sass_processor/management/commands/compilescss.py
action='store_true',
dest='exclude_py',
default=False,
help=_("Exclude python files from scan?")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nitpick: perhaps change question mark to dot.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Rather, no punctuation mark at all.

@jrief

jrief commented Oct 21, 2022

Copy link
Copy Markdown
Owner

@askvortsov1 which use-case does this feature have?

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.

4 participants