qtsass.importers.qss_importer() - Code Metrics - Inspection of "PR: Add support for compiling directories of QtSas..." - spyder-ide/qtsass - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#23)

unknown

created 2018-05-16 16:56 UTC

qtsass.importers.qss_importer() B

↳ Parent: qtsass.importers

Complexity

Conditions

Size

Total Lines	43
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	24
dl	0
loc	43
rs	7.5384
c	0
b	0
f	0
cc	6
nop	1

# -*- coding: utf-8 -*-
# -----------------------------------------------------------------------------
# Copyright (c) 2015 Yann Lanthony
# Copyright (c) 2017-2018 Spyder Project Contributors
#
# Licensed under the terms of the MIT License
# (See LICENSE.txt for details)
# -----------------------------------------------------------------------------
"""Libsass importers."""

# Standard library imports
from __future__ import absolute_import
import os

# Local imports
from qtsass.conformers import scss_conform


def norm_path(*parts):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return os.path.normpath(os.path.join(*parts))


def qss_importer(where):
    """
    Returns a function which conforms imported qss files to valid scss to be
    used as an importer for sass.compile.

    :param where: Directory containing scss, css, and sass files
    """

    def find_file(import_file):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

        # Create partial import filename
        dirname, basename = os.path.split(import_file)
        if dirname:
            import_partial_file = '/'.join([dirname, '_' + basename])
        else:
            import_partial_file = '_' + basename

        # Build potential file paths for @import "import_file"
        potential_files = []
        for ext in ['', '.scss', '.css', '.sass']:
            full_name = import_file + ext
            partial_name = import_partial_file + ext
            potential_files.append(full_name)
            potential_files.append(partial_name)
            potential_files.append(norm_path(where, full_name))
            potential_files.append(norm_path(where, partial_name))

        # Return first existing potential file
        for potential_file in potential_files:
            if os.path.isfile(potential_file):
                return potential_file

        return None

    def import_and_conform_file(import_file):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

        real_import_file = find_file(import_file)
        with open(real_import_file, 'r') as f:

            import_str = f.read()

        return [(import_file, scss_conform(import_str))]

    return import_and_conform_file


1			# -- coding: utf-8 --
2			# -----------------------------------------------------------------------------
3			# Copyright (c) 2015 Yann Lanthony
4			# Copyright (c) 2017-2018 Spyder Project Contributors
5			#
6			# Licensed under the terms of the MIT License
7			# (See LICENSE.txt for details)
8			# -----------------------------------------------------------------------------
9			"""Libsass importers."""
10
11			# Standard library imports
12			from __future__ import absolute_import
13			import os
14
15			# Local imports
16			from qtsass.conformers import scss_conform
17
18
19			def norm_path(*parts):
			0 ignored issues – show Coding Style introduced 2018-05-16 16:58 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
20			return os.path.normpath(os.path.join(*parts))
21
22
23			def qss_importer(where):
24			"""
25			Returns a function which conforms imported qss files to valid scss to be
26			used as an importer for sass.compile.
27
28			:param where: Directory containing scss, css, and sass files
29			"""
30
31			def find_file(import_file):
			0 ignored issues – show Coding Style introduced 2018-05-16 16:58 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
32
33			# Create partial import filename
34			dirname, basename = os.path.split(import_file)
35			if dirname:
36			import_partial_file = '/'.join([dirname, '_' + basename])
37			else:
38			import_partial_file = '_' + basename
39
40			# Build potential file paths for @import "import_file"
41			potential_files = []
42			for ext in ['', '.scss', '.css', '.sass']:
43			full_name = import_file + ext
44			partial_name = import_partial_file + ext
45			potential_files.append(full_name)
46			potential_files.append(partial_name)
47			potential_files.append(norm_path(where, full_name))
48			potential_files.append(norm_path(where, partial_name))
49
50			# Return first existing potential file
51			for potential_file in potential_files:
52			if os.path.isfile(potential_file):
53			return potential_file
54
55			return None
56
57			def import_and_conform_file(import_file):
			0 ignored issues – show Coding Style introduced 2018-05-08 02:01 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
58
59			real_import_file = find_file(import_file)
60			with open(real_import_file, 'r') as f:
			0 ignored issues – show Coding Style Naming introduced 2018-05-08 02:01 UTC by Report Bug Copy Issue Report The name `f` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
61			import_str = f.read()
62
63			return [(import_file, scss_conform(import_str))]
64
65			return import_and_conform_file
66

spyder-ide / qtsass

GitHub Access Token became invalid

Pull Request — master (#23)

qtsass.importers.qss_importer() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like