Skip to content

Fix CodeQL parsing errors by excluding Jekyll template files from JavaScript analysis #17

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.

Sign up for GitHub

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
sphildreth merged 3 commits into from
Dec 20, 2025
Merged

Fix CodeQL parsing errors by excluding Jekyll template files from JavaScript analysis #17

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9d09527
Initial plan
Copilot Dec 20, 2025
cb162da
Remove YAML front matter from JavaScript files to fix CodeQL parsing …
Copilot Dec 20, 2025
6496e4a
Add CodeQL config to exclude Jekyll template files from JavaScript an…
Copilot Dec 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions .github/codeql/codeql-config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# CodeQL configuration
# This file excludes Jekyll template files from JavaScript analysis
# to prevent parse errors from Liquid template syntax

name: "CodeQL config"

# Paths to exclude from all analyses
paths-ignore:
# Exclude Jekyll template JavaScript files that contain Liquid syntax
- 'docs/assets/js/main.js'
- 'docs/assets/js/search.js'
- 'melodee/docs/assets/js/main.js'
- 'melodee/docs/assets/js/search.js'
1 change: 1 addition & 0 deletions .github/workflows/codeql.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ jobs:
uses: github/codeql-action/init@v3
with:
languages: ${{ matrix.language }}
config-file: ./.github/codeql/codeql-config.yml
# If you wish to specify custom queries, you can do so here or in a config file.
# By default, queries listed here will override any specified in a config file.
# Prefix the list here with "+" to use these queries and those in the config file.
Expand Down
4 changes: 0 additions & 4 deletions docs/assets/js/main.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
---
exclude_in_search: true
layout: null
---
(function($) {
'use strict';
$(function() {
Expand Down
4 changes: 0 additions & 4 deletions docs/assets/js/search.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
---
layout: null
excluded_in_search: true
---
(function () {
function getQueryVariable(variable) {
var query = window.location.search.substring(1),
Expand Down
4 changes: 0 additions & 4 deletions melodee/docs/assets/js/main.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
---
exclude_in_search: true
layout: null
---
(function($) {
'use strict';
$(function() {
Expand Down
4 changes: 0 additions & 4 deletions melodee/docs/assets/js/search.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
---
layout: null
excluded_in_search: true
---
(function () {
function getQueryVariable(variable) {
var query = window.location.search.substring(1),
Expand Down