Skip to content

Commit

Permalink
Bump theme check action and fix theme check error (#3704)
Browse files Browse the repository at this point in the history
* Add .shopify to the .gitignore

* Fix a theme check error for invalid HTML

* Bump theme check action to v2

* Bump actions/checkout to v4
  • Loading branch information
graygilmore authored Jan 14, 2025
1 parent 4cd128f commit 2358467
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 5 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ jobs:
name: Lighthouse
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Lighthouse
uses: shopify/lighthouse-ci-action@v1
with:
Expand All @@ -19,8 +19,8 @@ jobs:
name: Theme Check
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Theme Check
uses: shopify/theme-check-action@v1
uses: shopify/theme-check-action@v2
with:
token: ${{ github.token }}
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,4 @@
.Trashes
ehthumbs.db
Thumbs.db
.shopify
12 changes: 10 additions & 2 deletions sections/header.liquid
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,15 @@
endfor
-%}

<{% if section.settings.sticky_header_type != 'none' %}sticky-header data-sticky-type="{{ section.settings.sticky_header_type }}"{% else %}div{% endif %} class="header-wrapper color-{{ section.settings.color_scheme }} gradient{% if section.settings.show_line_separator %} header-wrapper--border-bottom{% endif %}">
{% liquid
assign header_tag = 'div'

if section.settings.sticky_header_type != 'none'
assign header_tag = 'sticky-header'
endif
%}

<{{ header_tag }} {% if header_tag == 'sticky-header' %}data-sticky-type="{{ section.settings.sticky_header_type }}"{% endif %} class="header-wrapper color-{{ section.settings.color_scheme }} gradient{% if section.settings.show_line_separator %} header-wrapper--border-bottom{% endif %}">
{%- liquid
assign social_links = false
assign localization_forms = false
Expand Down Expand Up @@ -280,7 +288,7 @@
</a>
</div>
</header>
</{% if section.settings.sticky_header_type != 'none' %}sticky-header{% else %}div{% endif %}>
</{{ header_tag }}>

{%- if settings.cart_type == "notification" -%}
{%- render 'cart-notification', color_scheme: section.settings.color_scheme, desktop_menu_type: section.settings.menu_type_desktop -%}
Expand Down

0 comments on commit 2358467

Please sign in to comment.