d469f305de
* chore: 🤖 replace Rose::CodeEditor with Hds::CodeEditor - Remove CodeEditor 5 dependency * chore: 🤖 Replace CodeMirror5 dep remove CodeMirror5 and replace existing editors with HDS::CodeEditor * chore: 🤖 deleting old codeeditor styles * feat: 🎸 changing code editor changing code editor ✅ Closes: 0 * removed unused export * pr cleanup * updating tests for new code editor * responding to PR feedback * added whitespace * fixing linting violations * chore: 🤖 self-review PR self-review PR in the course of switching to new HDS code editor * fix: 🐛 fix linting bug * fix: 🐛 unneeded dependency remove jslint dependency as this is handled in the new code editor from HDS * fix: 🐛 Fixing import bug Fix import bug caused by importing scss modules that no longer exist * Fix and refactor code editor acceptance test * Fix and refactor code editor acceptance test * chore: 🤖 remove codemirror dependency * refactor: 💡 fix merge conflict issues * test: 💍 update e2e test selectors * test: 💍 fix target spec click event * refactor: 💡 clean up test selectors for code editor * refactor: 💡 clean up storage bucket test selectors * chore: 🤖 remove code mirror modifiers --------- Co-authored-by: calcaide <ccorvo@hashicorp.com> Co-authored-by: cameronperera <cameron.perera@hashicorp.com>
39 lines
1.4 KiB
JavaScript
39 lines
1.4 KiB
JavaScript
/**
|
|
* Copyright IBM Corp. 2021, 2026
|
|
* SPDX-License-Identifier: BUSL-1.1
|
|
*/
|
|
|
|
'use strict';
|
|
|
|
module.exports = {
|
|
extends: ['stylelint-config-standard-scss', 'stylelint-config-prettier-scss'],
|
|
rules: {
|
|
// We oftentimes want to have long form properties for readability
|
|
'declaration-block-no-redundant-longhand-properties': null,
|
|
|
|
// This is just a personal preference unless the quotes are needed
|
|
// to escape special characters
|
|
'function-url-quotes': null,
|
|
|
|
// This is often limited by the DOM and we want to order our declarations by our own standard
|
|
'no-descending-specificity': null,
|
|
|
|
// -Webkit animation CSS for older browsers. Need a decision to remove `-webkit` syntax in
|
|
// favor of normal CSS
|
|
'property-no-vendor-prefix': null,
|
|
'at-rule-no-vendor-prefix': null,
|
|
|
|
// As a team we want to stick to using kebab-case as our selector class pattern, but there are a couple
|
|
// of exceptions in which we overrride CSS from addons like HDS, in which they use
|
|
// pascal case and a double-kebab-case. This regex checks for all of those 3 patterns and won't fail
|
|
// lint check, but will fail if its any other pattern.
|
|
'selector-class-pattern': [
|
|
'(^([a-z][a-z0-9]*)(-{1,2}[a-z0-9]+)*$)',
|
|
{
|
|
message: (selector) =>
|
|
`Expected class selector "${selector}" to be kebab-case`,
|
|
},
|
|
],
|
|
},
|
|
};
|