Skip to content

Commit

Permalink
Merge pull request #26 from wkillerud/fix/crash-if-sassdoc-type-undef…
Browse files Browse the repository at this point in the history
…ined
  • Loading branch information
wkillerud authored Dec 22, 2022
2 parents 4431082 + bc24c8b commit 18189ba
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
12 changes: 6 additions & 6 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "some-sass",
"displayName": "Some Sass",
"description": "Provides code suggestions, documentation and code navigation for modern SCSS",
"version": "2.7.0",
"version": "2.7.1",
"publisher": "SomewhatStationery",
"license": "MIT",
"engines": {
Expand Down
2 changes: 1 addition & 1 deletion src/server/features/completion/completion-utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export function makeMixinDocumentation(symbol: ScssMixin): string {
* Use the SnippetString syntax to provide smart completions of parameter names.
*/
export function mapParameterSnippet(p: ScssParameter, index: number): string {
if (p.sassdoc?.type.length) {
if (p.sassdoc?.type?.length) {
const choices = parseStringLiteralChoices(p.sassdoc.type);
if (choices.length > 0) {
return `\${${index + 1}|${choices.join(",")}|}`;
Expand Down

0 comments on commit 18189ba

Please sign in to comment.