-
Notifications
You must be signed in to change notification settings - Fork 72
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (self-review)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 cc @gaetanmaisse
matrix: | ||
# Override automatic language detection by changing the below list | ||
# Supported options are ['csharp', 'cpp', 'go', 'java', 'javascript', 'python'] | ||
language: ['javascript'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typescript?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are you suggesting adding typescript?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good try @shilman but this repo only contains JS, plus:
Supported options are ['csharp', 'cpp', 'go', 'java', 'javascript', 'python']
I knew it! Deep down you're a TS fanboy. 😛 😂
matrix: | ||
# Override automatic language detection by changing the below list | ||
# Supported options are ['csharp', 'cpp', 'go', 'java', 'javascript', 'python'] | ||
language: ['javascript'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good try @shilman but this repo only contains JS, plus:
Supported options are ['csharp', 'cpp', 'go', 'java', 'javascript', 'python']
I knew it! Deep down you're a TS fanboy. 😛 😂
🚀 PR was released in |
This PR proposes adding CodeQL Analysis