Skip to content

update project description - fix code rendering #3397

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

Merged
merged 1 commit into from
Mar 24, 2018

Conversation

nlhkabu
Copy link
Contributor

@nlhkabu nlhkabu commented Mar 24, 2018

Closes #3330

screenshot from 2018-03-24 09-11-12

Also makes project release styles more robust - all elements now use margins at the top, for more consistent rendering.

@nlhkabu nlhkabu merged commit 0f92c18 into master Mar 24, 2018
@nlhkabu nlhkabu deleted the 3330-fix-project-description branch March 29, 2018 04:53
miketheman added a commit to miketheman/warehouse that referenced this pull request Jan 6, 2023
Introduced in pypi#3397

Raises lint error for `no-duplicate-selectors`.

- Remove duplication
- Disable lint rule for the Pygments block, since it helps us visually
  organize our thoughts.

Signed-off-by: Mike Fiedler <[email protected]>
di pushed a commit that referenced this pull request Jan 6, 2023
* chore(deps): install stylelint for SCSS

Signed-off-by: Mike Fiedler <[email protected]>

* lint: add stylelint configuration

Trying to match the existing `.sass-lint.yml` rules as much as we can.

Signed-off-by: Mike Fiedler <[email protected]>

* lint: address whitespace

Fixed based on the stylelint rules.

There's 65 remaining fixes if we choose to change enable:
- at-rule-empty-line-before
- declaration-empty-line-before

Signed-off-by: Mike Fiedler <[email protected]>

* lint: address pseudo selectors

With `selector-pseudo-element-colon-notation` enabled, we double-colon
pseudo selectors for visual specificity.

Signed-off-by: Mike Fiedler <[email protected]>

* lint: rename disables

Signed-off-by: Mike Fiedler <[email protected]>

* lint: handle shorthand properties

Refs: https://stylelint.io/user-guide/rules/shorthand-property-no-redundant-values/

Signed-off-by: Mike Fiedler <[email protected]>

* lint: simplify complex .not notation

Refs: https://stylelint.io/user-guide/rules/selector-not-notation/

Signed-off-by: Mike Fiedler <[email protected]>

* lint: handle color lints

Solves:

- alpha-value-notation: 5
- color-function-notation: 5
- color-hex-case: 3
- color-hex-length: 1

Signed-off-by: Mike Fiedler <[email protected]>

* lint: handle various one-off cases

- declaration-block-trailing-semicolon: 2
- font-family-name-quotes: 1
- length-zero-no-unit: 1
- number-leading-zero: 1
- property-no-vendor-prefix: 1
- value-keyword-case: 1

Signed-off-by: Mike Fiedler <[email protected]>

* lint: remove duplicate selectors

Introduced in #3397

Raises lint error for `no-duplicate-selectors`.

- Remove duplication
- Disable lint rule for the Pygments block, since it helps us visually
  organize our thoughts.

Signed-off-by: Mike Fiedler <[email protected]>

* refactor: replace my janky regex with plugin

Instead of trying to decipher in the future how I came up with the
selector pattern, use a plugin that implements the right selector
syntaxes.

We disable the inherited `selector-class-pattern` rule from
`stylelint-config-standard`.

Refs: stylelint/stylelint-config-standard#209

Signed-off-by: Mike Fiedler <[email protected]>

* test: run stylelint instead of sass-lint

Signed-off-by: Mike Fiedler <[email protected]>

* chore: remove deprecated sass-lint

Signed-off-by: Mike Fiedler <[email protected]>

Signed-off-by: Mike Fiedler <[email protected]>
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.

2 participants