Compare commits
1 Commits
master
...
first-time
Author | SHA1 | Date |
---|---|---|
Frank Taillandier | 7be44391de |
|
@ -1,103 +0,0 @@
|
|||
name: Continuous Integration
|
||||
|
||||
on:
|
||||
push:
|
||||
branches:
|
||||
- master
|
||||
pull_request:
|
||||
branches:
|
||||
- master
|
||||
|
||||
jobs:
|
||||
j4:
|
||||
if: "!contains(github.event.commits[0].message, '[ci skip]')"
|
||||
name: "Jekyll ${{ matrix.jekyll_version }} (Ruby ${{ matrix.ruby_version }})"
|
||||
runs-on: 'ubuntu-latest'
|
||||
env:
|
||||
JEKYLL_VERSION: ${{ matrix.jekyll_version }}
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
ruby_version:
|
||||
- 2.5
|
||||
- 2.7
|
||||
- 3.1
|
||||
jekyll_version:
|
||||
- "~> 4.0"
|
||||
steps:
|
||||
- uses: actions/checkout@v2
|
||||
with:
|
||||
fetch-depth: 5
|
||||
- name: "Set up Ruby ${{ matrix.ruby_version }}"
|
||||
uses: ruby/setup-ruby@v1
|
||||
with:
|
||||
ruby-version: ${{ matrix.ruby_version }}
|
||||
bundler-cache: true
|
||||
- name: Execute tests
|
||||
run: bundle exec rspec
|
||||
j3:
|
||||
if: "!contains(github.event.commits[0].message, '[ci skip]')"
|
||||
name: "Jekyll ${{ matrix.jekyll_version }} (Ruby ${{ matrix.ruby_version }})"
|
||||
runs-on: 'ubuntu-latest'
|
||||
env:
|
||||
JEKYLL_VERSION: ${{ matrix.jekyll_version }}
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
ruby_version:
|
||||
- 2.5
|
||||
jekyll_version:
|
||||
- "~> 3.9"
|
||||
steps:
|
||||
- uses: actions/checkout@v2
|
||||
with:
|
||||
fetch-depth: 5
|
||||
- name: "Set up Ruby ${{ matrix.ruby_version }}"
|
||||
uses: ruby/setup-ruby@v1
|
||||
with:
|
||||
ruby-version: ${{ matrix.ruby_version }}
|
||||
bundler-cache: true
|
||||
- name: Execute tests
|
||||
run: bundle exec rspec
|
||||
|
||||
style_check:
|
||||
if: "!contains(github.event.commits[0].message, '[ci skip]')"
|
||||
name: "Code Style Check (Ruby ${{ matrix.ruby_version }})"
|
||||
runs-on: 'ubuntu-latest'
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
ruby_version:
|
||||
- 2.5
|
||||
steps:
|
||||
- uses: actions/checkout@v2
|
||||
with:
|
||||
fetch-depth: 5
|
||||
- name: "Set up Ruby ${{ matrix.ruby_version }}"
|
||||
uses: ruby/setup-ruby@v1
|
||||
with:
|
||||
ruby-version: ${{ matrix.ruby_version }}
|
||||
bundler-cache: true
|
||||
- name: Check Style Offenses
|
||||
run: bundle exec rubocop -S -D
|
||||
|
||||
gem_build:
|
||||
if: "!contains(github.event.commits[0].message, '[ci skip]')"
|
||||
name: "Test Gem build (Ruby ${{ matrix.ruby_version }})"
|
||||
runs-on: 'ubuntu-latest'
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
ruby_version:
|
||||
- 2.5
|
||||
steps:
|
||||
- uses: actions/checkout@v2
|
||||
with:
|
||||
fetch-depth: 5
|
||||
- name: "Set up Ruby ${{ matrix.ruby_version }}"
|
||||
uses: ruby/setup-ruby@v1
|
||||
with:
|
||||
ruby-version: ${{ matrix.ruby_version }}
|
||||
bundler-cache: true
|
||||
- name: Test Gem build
|
||||
run: bundle exec gem build jekyll-seo-tag.gemspec
|
|
@ -1,33 +0,0 @@
|
|||
name: Release Gem
|
||||
|
||||
on:
|
||||
push:
|
||||
branches:
|
||||
- master
|
||||
paths:
|
||||
- "lib/**/version.rb"
|
||||
|
||||
jobs:
|
||||
release:
|
||||
if: "github.repository_owner == 'jekyll'"
|
||||
name: "Release Gem (Ruby ${{ matrix.ruby_version }})"
|
||||
runs-on: "ubuntu-latest"
|
||||
strategy:
|
||||
fail-fast: true
|
||||
matrix:
|
||||
ruby_version:
|
||||
- 2.7
|
||||
steps:
|
||||
- name: Checkout Repository
|
||||
uses: actions/checkout@v2
|
||||
- name: "Set up Ruby ${{ matrix.ruby_version }}"
|
||||
uses: ruby/setup-ruby@v1
|
||||
with:
|
||||
ruby-version: ${{ matrix.ruby_version }}
|
||||
bundler-cache: true
|
||||
- name: Build and Publish Gem
|
||||
uses: ashmaroli/release-gem@dist
|
||||
with:
|
||||
gemspec_name: jekyll-seo-tag
|
||||
env:
|
||||
GEM_HOST_API_KEY: ${{ secrets.RUBYGEMS_GEM_PUSH_API_KEY }}
|
|
@ -1,4 +0,0 @@
|
|||
#!/usr/bin/env bash
|
||||
|
||||
bundle exec ruby jekyll-seo-tag/.github/workflows/scripts/memprof.rb sandbox
|
||||
exit 0
|
|
@ -1,14 +0,0 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
require 'jekyll'
|
||||
require 'memory_profiler'
|
||||
|
||||
MemoryProfiler.report(allow_files: 'lib/jekyll-seo-tag') do
|
||||
Jekyll::PluginManager.require_from_bundler
|
||||
Jekyll::Commands::Build.process({
|
||||
"source" => File.expand_path(ARGV[0]),
|
||||
"destination" => File.expand_path("#{ARGV[0]}/_site"),
|
||||
"disable_disk_cache" => true,
|
||||
})
|
||||
puts ''
|
||||
end.pretty_print(scale_bytes: true, normalize_paths: true)
|
|
@ -1,63 +0,0 @@
|
|||
name: Third-Party Repository Profiling
|
||||
|
||||
on:
|
||||
push:
|
||||
branches:
|
||||
- master
|
||||
pull_request:
|
||||
branches:
|
||||
- master
|
||||
jobs:
|
||||
build_n_profile:
|
||||
name: "Third-Party Repo Profile (Jekyll ${{ matrix.jekyll_version }}, Ruby ${{ matrix.ruby_version }})"
|
||||
runs-on: "ubuntu-latest"
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
ruby_version:
|
||||
- 2.7
|
||||
jekyll_version:
|
||||
- "~> 4.0"
|
||||
- "~> 3.9"
|
||||
env:
|
||||
BUNDLE_GEMFILE: "sandbox/Gemfile"
|
||||
BUNDLE_PATH: "vendor/bundle"
|
||||
BUNDLE_JOBS: 4
|
||||
BUNDLE_RETRY: 3
|
||||
JEKYLL_VERSION: ${{ matrix.jekyll_version }}
|
||||
steps:
|
||||
- name: Checkout Jekyll SEO Tag
|
||||
uses: actions/checkout@v2
|
||||
with:
|
||||
fetch-depth: 5
|
||||
path: jekyll-seo-tag
|
||||
- name: Checkout Third-Party Repository (WITHOUT SEO Tag)
|
||||
uses: actions/checkout@v2
|
||||
with:
|
||||
repository: ashmaroli/tomjoht.github.io
|
||||
ref: "no-seo-tag"
|
||||
path: alpha-sandbox
|
||||
- name: Checkout Same Third-Party Repository (WITH SEO Tag)
|
||||
uses: actions/checkout@v2
|
||||
with:
|
||||
repository: ashmaroli/tomjoht.github.io
|
||||
ref: "seo-tag"
|
||||
path: sandbox
|
||||
- name: "Set up Ruby ${{ matrix.ruby_version }}"
|
||||
uses: ruby/setup-ruby@v1
|
||||
with:
|
||||
ruby-version: ${{ matrix.ruby_version }}
|
||||
bundler-cache: true
|
||||
- name: Run Jekyll Build (WITHOUT SEO Tag) 3 times
|
||||
run: |
|
||||
bundle exec jekyll build -s alpha-sandbox -d alpha-sandbox/_site --trace
|
||||
bundle exec jekyll build -s alpha-sandbox -d alpha-sandbox/_site --trace
|
||||
bundle exec jekyll build -s alpha-sandbox -d alpha-sandbox/_site --trace
|
||||
- name: Run Jekyll Build (WITH SEO Tag) 3 times
|
||||
run: |
|
||||
bundle exec jekyll build -s sandbox -d sandbox/_site --trace
|
||||
bundle exec jekyll build -s sandbox -d sandbox/_site --trace
|
||||
bundle exec jekyll build -s sandbox -d sandbox/_site --trace
|
||||
- name: Memory Analysis of Jekyll Build (WITH SEO Tag)
|
||||
run: bash jekyll-seo-tag/.github/workflows/scripts/memprof
|
||||
if: "!contains(matrix.jekyll_version, '3.')"
|
|
@ -1,9 +1,12 @@
|
|||
/.bundle/
|
||||
/.yardoc
|
||||
/Gemfile.lock
|
||||
/_yardoc/
|
||||
/coverage/
|
||||
/doc/
|
||||
/pkg/
|
||||
/spec/reports/
|
||||
/tmp/
|
||||
/bin/
|
||||
*.gem
|
||||
.bundle
|
||||
.yardoc
|
||||
Gemfile.lock
|
||||
pkg
|
||||
spec/reports/
|
||||
spec/fixtures/.jekyll-cache
|
||||
tmp/
|
||||
vendor/bundle
|
||||
_site
|
||||
|
|
41
.rubocop.yml
41
.rubocop.yml
|
@ -1,42 +1,25 @@
|
|||
inherit_from: .rubocop_todo.yml
|
||||
|
||||
require: rubocop-jekyll
|
||||
inherit_gem:
|
||||
rubocop-jekyll: .rubocop.yml
|
||||
jekyll: .rubocop.yml
|
||||
|
||||
AllCops:
|
||||
TargetRubyVersion: 2.5
|
||||
SuggestExtensions: false
|
||||
Exclude:
|
||||
- vendor/**/*
|
||||
|
||||
Layout/LineEndStringConcatenationIndentation:
|
||||
Enabled: true
|
||||
Layout/LineLength:
|
||||
Metrics/LineLength:
|
||||
Exclude:
|
||||
- spec/**/*
|
||||
- jekyll-seo-tag.gemspec
|
||||
|
||||
Lint/EmptyInPattern:
|
||||
Enabled: false
|
||||
|
||||
Metrics/BlockLength:
|
||||
Exclude:
|
||||
- spec/**/*
|
||||
|
||||
Naming/InclusiveLanguage:
|
||||
Style/Documentation:
|
||||
Enabled: false
|
||||
|
||||
Performance/MapCompact:
|
||||
Enabled: true
|
||||
Performance/RedundantEqualityComparisonBlock:
|
||||
Enabled: true
|
||||
Performance/RedundantSplitRegexpArgument:
|
||||
Enabled: true
|
||||
Style/FileName:
|
||||
Enabled: false
|
||||
|
||||
Style/InPatternThen:
|
||||
Enabled: false
|
||||
Style/MultilineInPatternThen:
|
||||
Enabled: false
|
||||
Style/QuotedSymbols:
|
||||
Enabled: true
|
||||
Style/IndentHeredoc:
|
||||
Exclude:
|
||||
- spec/**/*
|
||||
|
||||
AllCops:
|
||||
Exclude:
|
||||
- vendor/**/*
|
||||
|
|
|
@ -1,46 +0,0 @@
|
|||
# This configuration was generated by
|
||||
# `rubocop --auto-gen-config --auto-gen-only-exclude`
|
||||
# on 2021-09-17 06:40:32 UTC using RuboCop version 1.18.4.
|
||||
# The point is for the user to remove these configuration records
|
||||
# one by one as the offenses are removed from the code base.
|
||||
# Note that changes in the inspected code, or installation of new
|
||||
# versions of RuboCop, may require this file to be generated again.
|
||||
|
||||
# Offense count: 1
|
||||
# Configuration parameters: AllowComments.
|
||||
Lint/EmptyClass:
|
||||
Exclude:
|
||||
- 'lib/jekyll-seo-tag/version.rb'
|
||||
|
||||
# Offense count: 3
|
||||
Lint/NoReturnInBeginEndBlocks:
|
||||
Exclude:
|
||||
- 'lib/jekyll-seo-tag/author_drop.rb'
|
||||
- 'lib/jekyll-seo-tag/drop.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Lint/ToJSON:
|
||||
Exclude:
|
||||
- 'lib/jekyll-seo-tag/json_ld_drop.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Configuration parameters: IgnoredMethods, Max.
|
||||
Metrics/PerceivedComplexity:
|
||||
Exclude:
|
||||
- 'lib/jekyll-seo-tag/drop.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Configuration parameters: MinSize.
|
||||
Performance/CollectionLiteralInLoop:
|
||||
Exclude:
|
||||
- 'spec/jekyll_seo_tag/author_drop_spec.rb'
|
||||
|
||||
# Offense count: 9
|
||||
# Cop supports --auto-correct.
|
||||
Style/RedundantBegin:
|
||||
Exclude:
|
||||
- 'lib/jekyll-seo-tag.rb'
|
||||
- 'lib/jekyll-seo-tag/author_drop.rb'
|
||||
- 'lib/jekyll-seo-tag/drop.rb'
|
||||
- 'lib/jekyll-seo-tag/image_drop.rb'
|
|
@ -0,0 +1,7 @@
|
|||
rvm:
|
||||
- 2.2.3
|
||||
before_install: gem install bundler
|
||||
language: ruby
|
||||
script: script/cibuild
|
||||
sudo: false
|
||||
cache: bundler
|
5
Gemfile
5
Gemfile
|
@ -1,8 +1,5 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
source "https://rubygems.org"
|
||||
|
||||
gemspec
|
||||
|
||||
gem "jekyll", ENV["JEKYLL_VERSION"] if ENV["JEKYLL_VERSION"]
|
||||
gem "kramdown-parser-gfm" if ENV["JEKYLL_VERSION"] == "~> 3.9"
|
||||
gem "github-pages", :group => :jekyll_plugins
|
||||
|
|
178
History.markdown
178
History.markdown
|
@ -1,186 +1,40 @@
|
|||
## HEAD
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* Add Ruby 3.1 to CI matrix (#459)
|
||||
* chore: remove git.io (#462)
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Added the `twitter:description` tag (#464)
|
||||
* Support for image paths relative to the page's directory (#466)
|
||||
* Truncate the description value to 100 words (#492)
|
||||
|
||||
## 2.8.0 / 2022-02-04
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Allow to set type for author (#427)
|
||||
* Allow setting `author.url` (#453)
|
||||
* Implement Facebook domain verification (#455)
|
||||
* Add `og:image:alt` and `twitter:image:alt` (#438)
|
||||
* Sort JSON-LD data by key (#458)
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* Set the default `og:type` to 'website' (#391)
|
||||
* Template: Remove double new line (#454)
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* Fix typo in source code comment (#449)
|
||||
* Set up Continuous Integration via GH Actions (#450)
|
||||
* Bump RuboCop to v1.18.x (#452)
|
||||
* Add workflow to release gem via GH Actions
|
||||
|
||||
## 2.7.1 / 2020-10-18
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* refactor: mutate site payload instead of duplicating it (#419)
|
||||
|
||||
## 2.7.0 / 2020-10-18
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Change pagination message with `seo_paginator_message` option (#324)
|
||||
* Make Twitter Summary Card without having Twitter account (#284)
|
||||
* Prefer site.tagline to site.description for page title (#356)
|
||||
* Render og:locale meta only when defined explicitly (#388)
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* Ensure a single leading `@` for twitter usernames (#367)
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* chore(deps): require Ruby > 2.4.0 EOL
|
||||
* test: fix locale specs that use the fallback locale (#360)
|
||||
* refactor: Replace read-only empty hash with private constant (#418)
|
||||
* refactor: Mutate hash literals instead of duplicating them (#417)
|
||||
* refactor: Reduce allocations of instance-agnostic objects (#376)
|
||||
* refactor: Memoize #author_hash in SeoTag::AuthorDrop (#342)
|
||||
* refactor: simplify conditional in SeoTag::Drop#date_modified (#343)
|
||||
* chore(ci): profile seo-tag plugin on a third-party repository (#414)
|
||||
* chore(ci): Jekyll v4.0 (#372)
|
||||
* chore(ci): test against current stable Ruby 2.5 and 2.7 (#385)
|
||||
* style: align with latest jekyll-rubocop (#382)
|
||||
* fix: Travis builds for Jekyll 3.x (#415)
|
||||
|
||||
### Documentation
|
||||
|
||||
* Structured Data Testing Tool is deprecated (#409)
|
||||
* Rename Google webmaster tools to Google Search Console (#403)
|
||||
* Improve documentation on plugin usage (#399)
|
||||
* remove Google+ from example snippet (#358)
|
||||
* HTTPS link to https://ogp.me/ (#359)
|
||||
* HTTPS links to schema.org (#350)
|
||||
* use example.com for example URL (#351)
|
||||
|
||||
## 2.6.1 / 2019-05-17
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* Test against Jekyll 4.x (#336)
|
||||
|
||||
## 2.6.0 / 2019-03-16
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Twitter Image and Title (#330)
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* Do not cache the drop payload for SeoTag (#306)
|
||||
* Update url of schema website (#296)
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* Relax version constraint on Bundler (#325)
|
||||
* chore(ci): Add Ruby 2.6, drop Ruby 2.3 (#326)
|
||||
* chore (ci): remove deprecated `sudo: false` in .travis.yml (#333)
|
||||
* Lint Ruby code with rubocop-jekyll gem (#302)
|
||||
* chore(deps): bump rubocop-jekyll to v0.4 (#320)
|
||||
* chore(deps): bump rubocop-jekyll to v0.3 (#316)
|
||||
* Correct RuboCop offenses in spec files (#319)
|
||||
|
||||
### Documentation
|
||||
|
||||
* Rectify error in Usage documentation (#328)
|
||||
|
||||
## 2.5.0 / 2018-05-18
|
||||
|
||||
* Docs: Prevent GitHub Pages from processing Liquid raw tag (#276)
|
||||
|
||||
### Documentation
|
||||
|
||||
* Use gems config key for Jekyll < 3.5.0 (#255)
|
||||
* docs/usage - replace "below" with correct link (#280)
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* Test against Ruby 2.5 (#260)
|
||||
* add tests for twitter.card types (#289)
|
||||
* Target Ruby 2.3 and Rubocop 0.56.0 (#292)
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Add webmaster_verifications for baidu (#263)
|
||||
* Include page number in title (#250)
|
||||
* Configure default Twitter summary card type (V2) (#225)
|
||||
|
||||
## 2.4.0 / 2017-12-04
|
||||
|
||||
### Minor
|
||||
|
||||
* Add meta generator (#236)
|
||||
* Consistently use self-closing tags (#246)
|
||||
* Strip null values from JSON-LD hash (#249)
|
||||
|
||||
### Documentation
|
||||
|
||||
* Avoid deprecation warning when building docs (#243)
|
||||
|
||||
### Development Fixes
|
||||
|
||||
* Test against latest Rubies (#242)
|
||||
* Use Nokigiri on CI (#181)
|
||||
|
||||
## 2.3.0
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Use canonical_url specified in page if present #211
|
||||
* Fix for image.path causing an invalid url error #228
|
||||
* Ensure `site.data.authors` is properly formatted before attempting to retrieve author meta #227
|
||||
* Convert author, image, and JSON-LD to dedicated drops #229
|
||||
* Cache parsed template #231
|
||||
* Define path with `__dir__` #232
|
||||
* Use canonical_url specified in page if present #211
|
||||
* Fix for image.path causing an invalid url error #228
|
||||
* Ensure `site.data.authors` is properly formatted before attempting to retrieve author meta #227
|
||||
* Convert author, image, and JSON-LD to dedicated drops #229
|
||||
* Cache parsed template #231
|
||||
* Define path with `__dir__` #232
|
||||
|
||||
### Documentation
|
||||
|
||||
* gems: is deprecated in current Jekyll version of github-pages #230
|
||||
* gems: is deprecated in current Jekyll version of github-pages #230
|
||||
|
||||
## 2.2.3
|
||||
## 2.2.3
|
||||
|
||||
* Guard against the author's Twitter handle being Nil when stripping @'s #203
|
||||
* Guard against empty title or description strings #206
|
||||
* Guard against the author's Twitter handle being Nil when stripping @'s #203
|
||||
* Guard against empty title or description strings #206
|
||||
|
||||
## 2.2.2
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Guard against arrays in subhashes #197
|
||||
* Guard against invalid or missing URLs #199
|
||||
* Guard against arrays in subhashes #197
|
||||
* Guard against invalid or missing URLs #199
|
||||
|
||||
### Development fixes
|
||||
|
||||
* Remove dynamic GitHub Pages logic from Gemfile #194
|
||||
* Remove dynamic GitHub Pages logic from Gemfile #194
|
||||
|
||||
## 2.2.1
|
||||
|
||||
* Convert template logic to a Liquid Drop (significant performance improvement) (#184)
|
||||
* Fix for JSON-LD validation warning for images missing required properties (#183)
|
||||
* Convert template logic to a Liquid Drop (significant performance improvement) (#184)
|
||||
* Fix for JSON-LD validation warning for images missing required properties (#183)
|
||||
|
||||
## 2.2.0
|
||||
|
||||
|
@ -190,7 +44,7 @@
|
|||
* Add og:locale support #166
|
||||
* Add support for Bing and Yandex webmaster tools. Closes #147 (#148)
|
||||
* Add SEO author and date modified to validate JSON-LD output (#151)
|
||||
|
||||
|
||||
### Minor Enhancements
|
||||
|
||||
* Use `|` for title separator (#162)
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
The MIT License (MIT)
|
||||
|
||||
Copyright (c) 2015-present Ben Balter and the jekyll-seo-tag contributors
|
||||
Copyright (c) 2015 Ben Balter
|
||||
|
||||
Permission is hereby granted, free of charge, to any person obtaining a copy
|
||||
of this software and associated documentation files (the "Software"), to deal
|
||||
|
|
|
@ -12,8 +12,8 @@ Jekyll SEO Tag adds the following meta tags to your site:
|
|||
* Page description
|
||||
* Canonical URL
|
||||
* Next and previous URLs on paginated pages
|
||||
* [JSON-LD Site and post metadata](https://developers.google.com/search/docs/guides/intro-structured-data) for richer indexing
|
||||
* [Open Graph](https://ogp.me/) title, description, site title, and URL (for Facebook, LinkedIn, etc.)
|
||||
* [JSON-LD Site and post metadata](https://developers.google.com/structured-data/) for richer indexing
|
||||
* [Open Graph](http://ogp.me/) title, description, site title, and URL (for Facebook, LinkedIn, etc.)
|
||||
* [Twitter Summary Card](https://dev.twitter.com/cards/overview) metadata
|
||||
|
||||
While you could theoretically add the necessary metadata tags yourself, Jekyll SEO Tag provides a battle-tested template of crowdsourced best-practices.
|
||||
|
|
|
@ -3,7 +3,7 @@ description: A Jekyll plugin to add metadata tags for search engines and social
|
|||
|
||||
permalink: pretty
|
||||
|
||||
plugins:
|
||||
gems:
|
||||
- jekyll-seo-tag
|
||||
- jekyll-sitemap
|
||||
|
||||
|
|
|
@ -6,11 +6,9 @@ Jekyll SEO Tag is designed to implement SEO best practices by default and to be
|
|||
|
||||
If for some reason, you don't want the plugin to output `<title>` tags on each page, simply invoke the plugin within your template like so:
|
||||
|
||||
<!-- {% raw %} -->
|
||||
```
|
||||
{% seo title=false %}
|
||||
```
|
||||
<!-- {% endraw %} -->
|
||||
|
||||
### Author information
|
||||
|
||||
|
@ -68,49 +66,14 @@ There are several ways to convey this author-specific information. Author inform
|
|||
author: benbalter
|
||||
```
|
||||
|
||||
#### Setting author url
|
||||
|
||||
Starting from August 6, 2021 [Google recommends](https://developers.google.com/search/updates) to set the `author.url` property. This property helps Google to disambiguate the correct author of the article.
|
||||
|
||||
You can set it the same way as the other author properties. For example, you can put it in an `author` object, in the site's `_config.yml`, e.g.:
|
||||
|
||||
```yml
|
||||
author:
|
||||
name: My Name
|
||||
url: https://example.com/
|
||||
```
|
||||
|
||||
### Customizing description length
|
||||
|
||||
By default, the description is limited to the first 100 words of the full content.
|
||||
|
||||
You can adjust this limit at the page level, by using the `seo_description_max_words` page property:
|
||||
|
||||
```yml
|
||||
seo_description_max_words: 200
|
||||
```
|
||||
|
||||
You can also set a default site-wide value for all pages using [Front Matter defaults](https://jekyllrb.com/docs/configuration/front-matter-defaults/) in your `_config.yml` file:
|
||||
|
||||
```yml
|
||||
defaults:
|
||||
- scope:
|
||||
path: ""
|
||||
values:
|
||||
seo_description_max_words: 200
|
||||
```
|
||||
|
||||
|
||||
### Customizing JSON-LD output
|
||||
|
||||
The following options can be set for any particular page. While the default options are meant to serve most users in the most common circumstances, there may be situations where more precise control is necessary.
|
||||
|
||||
* `seo`
|
||||
* `name` - If the name of the thing that the page represents is different from the page title. (i.e.: "Frank's Café" vs "Welcome to Frank's Café")
|
||||
* `type` - The type of things that the page represents. This must be a [Schema.org type](https://schema.org/docs/schemas.html), and will probably usually be something like [`BlogPosting`](https://schema.org/BlogPosting), [`NewsArticle`](https://schema.org/NewsArticle), [`Person`](https://schema.org/Person), [`Organization`](https://schema.org/Organization), etc.
|
||||
* `type` - The type of things that the page represents. This must be a [Schema.org type](http://schema.org/docs/schemas.html), and will probably usually be something like [`BlogPosting`](http://schema.org/BlogPosting), [`NewsArticle`](http://schema.org/NewsArticle), [`Person`](http://schema.org/Person), [`Organization`](http://schema.org/Organization), etc.
|
||||
* `links` - An array of other URLs that represent the same thing that this page represents. For instance, Jane's bio page might include links to Jane's GitHub and Twitter profiles.
|
||||
* `date_modified` - Manually specify the `dateModified` field in the JSON-LD output to override Jekyll's own `dateModified`.
|
||||
This field will take **first priority** for the `dateModified` JSON-LD output. This is useful when the file timestamp does not match the true time that the content was modified. A user may also install [Last Modified At](https://github.com/gjtorikian/jekyll-last-modified-at) which will offer an alternative way of providing for the `dateModified` field.
|
||||
|
||||
### Customizing image output
|
||||
|
||||
|
@ -119,7 +82,6 @@ For most users, setting `image: [path-to-image]` on a per-page basis should be e
|
|||
* `path` - The relative path to the image. Same as `image: [path-to-image]`
|
||||
* `height` - The height of the Open Graph (`og:image`) image
|
||||
* `width` - The width of the Open Graph (`og:image`) image
|
||||
* `alt` - The alternative image text for Open Graph (`og:image:alt`) and Twitter (`twitter:image:alt`)
|
||||
|
||||
You can use any of the above, optional properties, like so:
|
||||
|
||||
|
@ -128,12 +90,11 @@ image:
|
|||
path: /img/twitter.png
|
||||
height: 100
|
||||
width: 100
|
||||
alt: Twitter Logo
|
||||
```
|
||||
|
||||
### Setting a default image
|
||||
|
||||
You can define a default image using [Front Matter defaults](https://jekyllrb.com/docs/configuration/front-matter-defaults/), to provide a default Twitter Card or OGP image to all of your posts and pages.
|
||||
You can define a default image using [Front Matter default](https://jekyllrb.com/docs/configuration/#front-matter-defaults), to provide a default Twitter Card or OGP image to all of your posts and pages.
|
||||
|
||||
Here is a very basic example, that you are encouraged to adapt to your needs:
|
||||
|
||||
|
@ -147,11 +108,11 @@ defaults:
|
|||
|
||||
### SmartyPants Titles
|
||||
|
||||
Titles will be processed using [Jekyll's `smartify` filter](https://jekyllrb.com/docs/liquid/filters/). This will use SmartyPants to translate plain ASCII punctuation into "smart" typographic punctuation. This will not render or strip any Markdown you may be using in a page title.
|
||||
Titles will be processed using [Jekyll's `smartify` filter](https://jekyllrb.com/docs/templates/). This will use SmartyPants to translate plain ASCII punctuation into "smart" typographic punctuation. This will not render or strip any Markdown you may be using in a page title.
|
||||
|
||||
### Setting customized Canonical URL
|
||||
|
||||
You can set custom Canonical URL for a page by specifying canonical_url option in page front matter.
|
||||
You can set custom Canonical URL for a page by specifying canonical_url option in page front-matter.
|
||||
E.g., you have the following in the page's front matter:
|
||||
```yml
|
||||
layout: post
|
||||
|
@ -165,7 +126,7 @@ Which will generate canonical_url with specified link in canonical_url.
|
|||
```
|
||||
|
||||
If no canonical_url option was specified, then uses page url for generating canonical_url.
|
||||
E.g., you have not specified canonical_url in front matter:
|
||||
E.g., you have not specified canonical_url in front-matter:
|
||||
```yml
|
||||
layout: post
|
||||
title: Title of Your Post
|
||||
|
@ -173,19 +134,5 @@ title: Title of Your Post
|
|||
|
||||
Which will generate following canonical_url:
|
||||
```html
|
||||
<link rel="canonical" href="https://example.com/title-of-your-post" />
|
||||
<link rel="canonical" href="http://yoursite.com/title-of-your-post" />
|
||||
```
|
||||
|
||||
### Customizing title modifier for paginated pages
|
||||
|
||||
You can override the default title modifier for paginated pages from `Page %{current} of %{total} for ` to a string of your
|
||||
choice by setting a `seo_paginator_message` key in your `_config.yml`.
|
||||
|
||||
For example:
|
||||
|
||||
```yml
|
||||
seo_paginator_message: "%<current>s / %<total>s | "
|
||||
```
|
||||
|
||||
While the value can be any string text, we recommend using a Ruby string-template containing the variables `current` and `total`
|
||||
similar to the example above, to incorporate the current page-number and total number of paginated pages in the title.
|
||||
|
|
|
@ -13,12 +13,10 @@
|
|||
- jekyll-seo-tag
|
||||
```
|
||||
|
||||
If you are using a Jekyll version less than `3.5.0`, use the `gems` key instead of `plugins`.
|
||||
:bulb: If you are using a Jekyll version less than `3.5.0`, use the `gems` key instead of `plugins`.
|
||||
|
||||
3. Add the following right before `</head>` in your site's template(s):
|
||||
|
||||
<!-- {% raw %} -->
|
||||
```liquid
|
||||
{% seo %}
|
||||
```
|
||||
<!-- {% endraw %} -->
|
||||
|
|
|
@ -1,29 +1,18 @@
|
|||
## Usage
|
||||
|
||||
The SEO tag will respect any of the following if included in your site's `_config.yml` (and simply not include them if
|
||||
they're not defined):
|
||||
The SEO tag will respect any of the following if included in your site's `_config.yml` (and simply not include them if they're not defined):
|
||||
|
||||
* `title` - Your site's title (e.g., *Ben's Awesome Site*, *The GitHub Blog*, etc.), used as part of the title tag like
|
||||
`Home | Ben's Awesome Site`.
|
||||
* `tagline` - A short description (e.g., *A blog dedicated to reviewing cat gifs*), used as part of the title tag like
|
||||
`Ben's Awesome Site | A blog dedicated to reviewing cat gifs` instead of `Ben's Awesome Site | Long description about a
|
||||
blog dedicated to reviewing cat gifs` that would be used when `page.title` is not defined.
|
||||
* `description` - A longer description used for the description meta tag. Also used as fallback for pages that don't
|
||||
provide their own `description`, and also as part of the page's title tag if neither `page.title` nor `site.tagline`
|
||||
has been defined.
|
||||
* `title` - Your site's title (e.g., Ben's awesome site, The GitHub Blog, etc.)
|
||||
* `description` - A short description (e.g., A blog dedicated to reviewing cat gifs)
|
||||
* `url` - The full URL to your site. Note: `site.github.url` will be used by default.
|
||||
* `author` - global author information (see [Advanced usage](advanced-usage.md#author-information))
|
||||
* `twitter` - The following properties are available:
|
||||
* `twitter:card` - The site's default card type
|
||||
* `twitter:username` - The site's Twitter handle.
|
||||
|
||||
You'll want to describe them like so:
|
||||
* `author` - global author information (see below)
|
||||
* `twitter:username` - The site's Twitter handle. You'll want to describe it like so:
|
||||
|
||||
```yml
|
||||
twitter:
|
||||
username: benbalter
|
||||
card: summary
|
||||
```
|
||||
|
||||
* `facebook` - The following properties are available:
|
||||
* `facebook:app_id` - a Facebook app ID for Facebook insights
|
||||
* `facebook:publisher` - a Facebook page URL or ID of the publishing entity
|
||||
|
@ -31,20 +20,18 @@ has been defined.
|
|||
|
||||
You'll want to describe one or more like so:
|
||||
|
||||
```yml
|
||||
```yml
|
||||
facebook:
|
||||
app_id: 1234
|
||||
publisher: 1234
|
||||
admins: 1234
|
||||
```
|
||||
* `logo` - URL to a site-wide logo (e.g., `/assets/your-company-logo.png`) - If you would like the "publisher" property
|
||||
to be present, you must add this field to your site's configuration, during the validation of the structured data by
|
||||
Google Search Console, if the `logo` field is not validated, you will find errors inherent to the publisher in the
|
||||
[Rich Results Testing Tool](https://search.google.com/test/rich-results)
|
||||
* `social` - For [specifying social profiles](https://developers.google.com/search/docs/guides/enhance-site#add-your-sites-name-logo-and-social-links).
|
||||
The following properties are available:
|
||||
```
|
||||
|
||||
* `logo` - URL to a site-wide logo (e.g., `/assets/your-company-logo.png`)
|
||||
* `social` - For [specifying social profiles](https://developers.google.com/structured-data/customize/social-profiles). The following properties are available:
|
||||
* `name` - If the user or organization name differs from the site's name
|
||||
* `links` - An array of links to social media profiles.
|
||||
* `date_modified` - Manually specify the `dateModified` field in the JSON-LD output to override Jekyll's own `dateModified`. This field will take **first priority** for the `dateModified` JSON-LD output. This is useful when the file timestamp does not match the true time that the content was modified. A user may also install [Last Modified At](https://github.com/gjtorikian/jekyll-last-modified-at) which will offer an alternative way of providing for the `dateModified` field.
|
||||
|
||||
```yml
|
||||
social:
|
||||
|
@ -53,29 +40,28 @@ The following properties are available:
|
|||
- https://twitter.com/BenBalter
|
||||
- https://www.facebook.com/ben.balter
|
||||
- https://www.linkedin.com/in/BenBalter
|
||||
- https://plus.google.com/+BenBalter
|
||||
- https://github.com/benbalter
|
||||
- https://keybase.io/benbalter
|
||||
```
|
||||
* `google_site_verification` for verifying ownership via Google Search Console
|
||||
```
|
||||
|
||||
* `google_site_verification` for verifying ownership via Google webmaster tools
|
||||
* Alternatively, verify ownership with several services at once using the following format:
|
||||
```yml
|
||||
webmaster_verifications:
|
||||
google: 1234
|
||||
bing: 1234
|
||||
alexa: 1234
|
||||
yandex: 1234
|
||||
baidu: 1234
|
||||
facebook: 1234
|
||||
```
|
||||
* `locale` - The locale these tags are marked up in. Of the format `language_TERRITORY`. Default is `en_US`. Takes priority
|
||||
over existing config key `lang`.
|
||||
|
||||
```yml
|
||||
webmaster_verifications:
|
||||
google: 1234
|
||||
bing: 1234
|
||||
alexa: 1234
|
||||
yandex: 1234
|
||||
```
|
||||
|
||||
* `lang` - The locale these tags are marked up in. Of the format `language_TERRITORY`. Default is `en_US`.
|
||||
|
||||
The SEO tag will respect the following YAML front matter if included in a post, page, or document:
|
||||
|
||||
* `title` - The title of the post, page, or document
|
||||
* `description` - A short description of the page's content
|
||||
* `image` - URL to an image associated with the post, page, or document (e.g., `/assets/page-pic.jpg`)
|
||||
* `author` - Page-, post-, or document-specific author information (see [Advanced usage](advanced-usage.md#author-information))
|
||||
* `locale` - Page-, post-, or document-specific locale information. Takes priority over existing front matter attribute `lang`.
|
||||
|
||||
*Note:* Front matter defaults can be used for any of the above values as described in advanced usage with an image example.
|
||||
* `author` - Page-, post-, or document-specific author information (see below)
|
||||
* `lang` - Page-, post-, or document-specific language information
|
||||
|
|
|
@ -1,6 +1,8 @@
|
|||
# frozen_string_literal: true
|
||||
# coding: utf-8
|
||||
|
||||
require_relative "lib/jekyll-seo-tag/version"
|
||||
lib = File.expand_path("lib", __dir__)
|
||||
$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
|
||||
require "jekyll-seo-tag/version"
|
||||
|
||||
Gem::Specification.new do |spec|
|
||||
spec.name = "jekyll-seo-tag"
|
||||
|
@ -8,7 +10,7 @@ Gem::Specification.new do |spec|
|
|||
spec.authors = ["Ben Balter"]
|
||||
spec.email = ["ben.balter@github.com"]
|
||||
spec.summary = "A Jekyll plugin to add metadata tags for search engines and social networks to better index and display your site's content."
|
||||
spec.homepage = "https://github.com/jekyll/jekyll-seo-tag"
|
||||
spec.homepage = "https://github.com/benbalter/jekyll-seo-tag"
|
||||
spec.license = "MIT"
|
||||
|
||||
# Prevent pushing this gem to RubyGems.org by setting 'allowed_push_host', or
|
||||
|
@ -19,16 +21,14 @@ Gem::Specification.new do |spec|
|
|||
raise "RubyGems 2.0 or newer is required to protect against public gem pushes."
|
||||
end
|
||||
|
||||
spec.required_ruby_version = ">= 2.5.0"
|
||||
|
||||
spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r!^(test|spec|features)/!) }
|
||||
spec.bindir = "exe"
|
||||
spec.executables = spec.files.grep(%r!^exe/!) { |f| File.basename(f) }
|
||||
spec.require_paths = ["lib"]
|
||||
|
||||
spec.add_dependency "jekyll", ">= 3.8", "< 5.0"
|
||||
spec.add_development_dependency "bundler", ">= 1.15"
|
||||
spec.add_development_dependency "html-proofer", "~> 3.7"
|
||||
spec.add_dependency "jekyll", "~> 3.3"
|
||||
spec.add_development_dependency "bundler", "~> 1.14"
|
||||
spec.add_development_dependency "rspec", "~> 3.5"
|
||||
spec.add_development_dependency "rubocop-jekyll", "~> 0.12.0"
|
||||
spec.add_development_dependency "html-proofer", "~> 3.6"
|
||||
spec.add_development_dependency "rubocop", "~> 0.48"
|
||||
end
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
require "jekyll"
|
||||
require "jekyll-seo-tag/version"
|
||||
|
||||
|
@ -24,7 +22,7 @@ module Jekyll
|
|||
# We will not strip any whitespace if the next character is a '-'
|
||||
# so that we do not interfere with the HTML comment at the
|
||||
# very begining
|
||||
MINIFY_REGEX = %r!(?<=[{}]|[>,]\n)\s+(?\!-)!.freeze
|
||||
MINIFY_REGEX = %r!(?<=[{}]|[>,]\n)\s+(?\!-)!
|
||||
|
||||
def initialize(_tag_name, text, _tokens)
|
||||
super
|
||||
|
@ -46,20 +44,16 @@ module Jekyll
|
|||
end
|
||||
|
||||
def payload
|
||||
# site_payload is an instance of UnifiedPayloadDrop. See https://github.com/jekyll/jekyll/blob/22f2724a1f117a94cc16d18c499a93d5915ede4f/lib/jekyll/site.rb#L261-L276
|
||||
context.registers[:site].site_payload.tap do |site_payload|
|
||||
site_payload["page"] = context.registers[:page]
|
||||
site_payload["paginator"] = context["paginator"]
|
||||
site_payload["seo_tag"] = drop
|
||||
end
|
||||
# site_payload is an instance of UnifiedPayloadDrop. See https://git.io/v5ajm
|
||||
Jekyll::Utils.deep_merge_hashes(context.registers[:site].site_payload, {
|
||||
"page" => context.registers[:page],
|
||||
"paginator" => context["paginator"],
|
||||
"seo_tag" => drop,
|
||||
})
|
||||
end
|
||||
|
||||
def drop
|
||||
if context.registers[:site].liquid_renderer.respond_to?(:cache)
|
||||
Jekyll::SeoTag::Drop.new(@text, @context)
|
||||
else
|
||||
@drop ||= Jekyll::SeoTag::Drop.new(@text, @context)
|
||||
end
|
||||
@drop ||= Jekyll::SeoTag::Drop.new(@text, @context)
|
||||
end
|
||||
|
||||
def info
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
module Jekyll
|
||||
class SeoTag
|
||||
# A drop representing the current page's author
|
||||
|
@ -19,7 +17,6 @@ module Jekyll
|
|||
# site - The Jekyll::Drops::SiteDrop
|
||||
def initialize(page: nil, site: nil)
|
||||
raise ArgumentError unless page && site
|
||||
|
||||
@mutations = {}
|
||||
@page = page
|
||||
@site = site
|
||||
|
@ -34,21 +31,20 @@ module Jekyll
|
|||
|
||||
def twitter
|
||||
return @twitter if defined? @twitter
|
||||
|
||||
twitter = author_hash["twitter"] || author_hash["name"]
|
||||
@twitter = twitter.is_a?(String) ? twitter.sub(%r!^@!, "") : nil
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
attr_reader :site, :page
|
||||
attr_reader :page
|
||||
attr_reader :site
|
||||
|
||||
# Finds the page author in the page.author, page.authors, or site.author
|
||||
#
|
||||
# Returns a string or hash representing the author
|
||||
def resolved_author
|
||||
return @resolved_author if defined? @resolved_author
|
||||
|
||||
sources = [page["author"]]
|
||||
sources << page["authors"].first if page["authors"].is_a?(Array)
|
||||
sources << site["author"]
|
||||
|
@ -63,7 +59,6 @@ module Jekyll
|
|||
@site_data_hash ||= begin
|
||||
return {} unless resolved_author.is_a?(String)
|
||||
return {} unless site.data["authors"].is_a?(Hash)
|
||||
|
||||
author_hash = site.data["authors"][resolved_author]
|
||||
author_hash.is_a?(Hash) ? author_hash : {}
|
||||
end
|
||||
|
@ -73,15 +68,12 @@ module Jekyll
|
|||
# including site-wide metadata if the author is provided as a string,
|
||||
# or an empty hash, if the author cannot be resolved
|
||||
def author_hash
|
||||
@author_hash ||= begin
|
||||
case resolved_author
|
||||
when Hash
|
||||
resolved_author
|
||||
when String
|
||||
{ "name" => resolved_author }.merge!(site_data_hash)
|
||||
else
|
||||
{}
|
||||
end
|
||||
if resolved_author.is_a? Hash
|
||||
resolved_author
|
||||
elsif resolved_author.is_a? String
|
||||
{ "name" => resolved_author }.merge(site_data_hash)
|
||||
else
|
||||
{}
|
||||
end
|
||||
end
|
||||
|
||||
|
|
|
@ -1,23 +1,18 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
module Jekyll
|
||||
class SeoTag
|
||||
class Drop < Jekyll::Drops::Drop
|
||||
include Jekyll::SeoTag::UrlHelper
|
||||
|
||||
TITLE_SEPARATOR = " | "
|
||||
FORMAT_STRING_METHODS = [
|
||||
:markdownify, :strip_html, :normalize_whitespace, :escape_once,
|
||||
TITLE_SEPARATOR = " | ".freeze
|
||||
FORMAT_STRING_METHODS = %i[
|
||||
markdownify strip_html normalize_whitespace escape_once
|
||||
].freeze
|
||||
HOMEPAGE_OR_ABOUT_REGEX = %r!^/(about/)?(index.html?)?$!.freeze
|
||||
|
||||
EMPTY_READ_ONLY_HASH = {}.freeze
|
||||
private_constant :EMPTY_READ_ONLY_HASH
|
||||
HOMEPAGE_OR_ABOUT_REGEX = %r!^/(about/)?(index.html?)?$!
|
||||
|
||||
def initialize(text, context)
|
||||
@obj = EMPTY_READ_ONLY_HASH
|
||||
@obj = {}
|
||||
@mutations = {}
|
||||
@text = text
|
||||
@text = text
|
||||
@context = context
|
||||
end
|
||||
|
||||
|
@ -29,7 +24,6 @@ module Jekyll
|
|||
def title?
|
||||
return false unless title
|
||||
return @display_title if defined?(@display_title)
|
||||
|
||||
@display_title = (@text !~ %r!title=false!i)
|
||||
end
|
||||
|
||||
|
@ -37,10 +31,6 @@ module Jekyll
|
|||
@site_title ||= format_string(site["title"] || site["name"])
|
||||
end
|
||||
|
||||
def site_tagline
|
||||
@site_tagline ||= format_string site["tagline"]
|
||||
end
|
||||
|
||||
def site_description
|
||||
@site_description ||= format_string site["description"]
|
||||
end
|
||||
|
@ -50,32 +40,21 @@ module Jekyll
|
|||
@page_title ||= format_string(page["title"]) || site_title
|
||||
end
|
||||
|
||||
def site_tagline_or_description
|
||||
site_tagline || site_description
|
||||
end
|
||||
|
||||
# Page title with site title or description appended
|
||||
# rubocop:disable Metrics/CyclomaticComplexity
|
||||
def title
|
||||
@title ||= begin
|
||||
if site_title && page_title != site_title
|
||||
page_title + TITLE_SEPARATOR + site_title
|
||||
elsif site_description && site_title
|
||||
site_title + TITLE_SEPARATOR + site_tagline_or_description
|
||||
site_title + TITLE_SEPARATOR + site_description
|
||||
else
|
||||
page_title || site_title
|
||||
end
|
||||
end
|
||||
|
||||
return page_number + @title if page_number
|
||||
|
||||
@title
|
||||
end
|
||||
# rubocop:enable Metrics/CyclomaticComplexity
|
||||
|
||||
def name
|
||||
return @name if defined?(@name)
|
||||
|
||||
@name = if seo_name
|
||||
seo_name
|
||||
elsif !homepage_or_about?
|
||||
|
@ -89,8 +68,7 @@ module Jekyll
|
|||
|
||||
def description
|
||||
@description ||= begin
|
||||
value = format_string(page["description"] || page["excerpt"]) || site_description
|
||||
snippet(value, description_max_words)
|
||||
format_string(page["description"] || page["excerpt"]) || site_description
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -113,7 +91,13 @@ module Jekyll
|
|||
|
||||
def date_modified
|
||||
@date_modified ||= begin
|
||||
date = page_seo["date_modified"] || page["last_modified_at"].to_liquid || page["date"]
|
||||
date = if page_seo["date_modified"]
|
||||
page_seo["date_modified"]
|
||||
elsif page["last_modified_at"]
|
||||
page["last_modified_at"].to_liquid
|
||||
else
|
||||
page["date"]
|
||||
end
|
||||
filters.date_to_xmlschema(date) if date
|
||||
end
|
||||
end
|
||||
|
@ -149,7 +133,6 @@ module Jekyll
|
|||
def logo
|
||||
@logo ||= begin
|
||||
return unless site["logo"]
|
||||
|
||||
if absolute_url? site["logo"]
|
||||
filters.uri_escape site["logo"]
|
||||
else
|
||||
|
@ -162,10 +145,6 @@ module Jekyll
|
|||
@page_lang ||= page["lang"] || site["lang"] || "en_US"
|
||||
end
|
||||
|
||||
def page_locale
|
||||
@page_locale ||= (page["locale"] || site["locale"] || page_lang).tr("-", "_")
|
||||
end
|
||||
|
||||
def canonical_url
|
||||
@canonical_url ||= begin
|
||||
if page["canonical_url"].to_s.empty?
|
||||
|
@ -176,10 +155,6 @@ module Jekyll
|
|||
end
|
||||
end
|
||||
|
||||
def description_max_words
|
||||
@description_max_words ||= page["seo_description_max_words"] || 100
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def filters
|
||||
|
@ -198,16 +173,6 @@ module Jekyll
|
|||
page["url"] =~ HOMEPAGE_OR_ABOUT_REGEX
|
||||
end
|
||||
|
||||
def page_number
|
||||
return unless @context["paginator"] && @context["paginator"]["page"]
|
||||
|
||||
current = @context["paginator"]["page"]
|
||||
total = @context["paginator"]["total_pages"]
|
||||
paginator_message = site["seo_paginator_message"] || "Page %<current>s of %<total>s for "
|
||||
|
||||
format(paginator_message, :current => current, :total => total) if current > 1
|
||||
end
|
||||
|
||||
attr_reader :context
|
||||
|
||||
def fallback_data
|
||||
|
@ -222,13 +187,6 @@ module Jekyll
|
|||
string unless string.empty?
|
||||
end
|
||||
|
||||
def snippet(string, max_words)
|
||||
return string if string.nil?
|
||||
|
||||
result = string.split(%r!\s+!, max_words + 1)[0...max_words].join(" ")
|
||||
result.length < string.length ? result.concat("…") : result
|
||||
end
|
||||
|
||||
def seo_name
|
||||
@seo_name ||= format_string(page_seo["name"]) if page_seo["name"]
|
||||
end
|
||||
|
@ -251,7 +209,7 @@ module Jekyll
|
|||
if hash[key].is_a?(Hash)
|
||||
hash[key]
|
||||
else
|
||||
EMPTY_READ_ONLY_HASH
|
||||
{}
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
module Jekyll
|
||||
class SeoTag
|
||||
class Filters
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
module Jekyll
|
||||
class SeoTag
|
||||
# A drop representing the page image
|
||||
|
@ -18,7 +16,6 @@ module Jekyll
|
|||
# context - the Liquid::Context
|
||||
def initialize(page: nil, context: nil)
|
||||
raise ArgumentError unless page && context
|
||||
|
||||
@mutations = {}
|
||||
@page = page
|
||||
@context = context
|
||||
|
@ -34,22 +31,18 @@ module Jekyll
|
|||
|
||||
private
|
||||
|
||||
attr_accessor :page, :context
|
||||
attr_accessor :page
|
||||
attr_accessor :context
|
||||
|
||||
# The normalized image hash with a `path` key (which may be nil)
|
||||
def image_hash
|
||||
@image_hash ||= begin
|
||||
image_meta = page["image"]
|
||||
|
||||
case image_meta
|
||||
when Hash
|
||||
{ "path" => nil }.merge!(image_meta)
|
||||
when String
|
||||
{ "path" => image_meta }
|
||||
else
|
||||
{ "path" => nil }
|
||||
end
|
||||
end
|
||||
@image_hash ||= if page["image"].is_a?(Hash)
|
||||
{ "path" => nil }.merge(page["image"])
|
||||
elsif page["image"].is_a?(String)
|
||||
{ "path" => page["image"] }
|
||||
else
|
||||
{ "path" => nil }
|
||||
end
|
||||
end
|
||||
alias_method :fallback_data, :image_hash
|
||||
|
||||
|
@ -61,18 +54,12 @@ module Jekyll
|
|||
|
||||
def absolute_url
|
||||
return unless raw_path
|
||||
|
||||
@absolute_url ||= build_absolute_path
|
||||
end
|
||||
|
||||
def build_absolute_path
|
||||
return raw_path unless raw_path.is_a?(String) && absolute_url?(raw_path) == false
|
||||
return filters.absolute_url(raw_path) if raw_path.start_with?("/")
|
||||
|
||||
page_dir = @page["url"]
|
||||
page_dir = File.dirname(page_dir) unless page_dir.end_with?("/")
|
||||
|
||||
filters.absolute_url File.join(page_dir, raw_path)
|
||||
return @absolute_url if defined? @absolute_url
|
||||
@absolute_url = if raw_path.is_a?(String) && absolute_url?(raw_path) == false
|
||||
filters.absolute_url raw_path
|
||||
else
|
||||
raw_path
|
||||
end
|
||||
end
|
||||
|
||||
def filters
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
module Jekyll
|
||||
class SeoTag
|
||||
# This module is deprecated, but is included in the Gem to avoid a breaking
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
module Jekyll
|
||||
class SeoTag
|
||||
class JSONLDDrop < Jekyll::Drops::Drop
|
||||
|
@ -16,12 +14,9 @@ module Jekyll
|
|||
def_delegator :page_drop, :type, :type
|
||||
|
||||
# Expose #type and #logo as private methods and #@type as a public method
|
||||
alias_method :@type, :type
|
||||
private :type, :logo
|
||||
|
||||
VALID_ENTITY_TYPES = %w(BlogPosting CreativeWork).freeze
|
||||
VALID_AUTHOR_TYPES = %w(Organization Person).freeze
|
||||
private_constant :VALID_ENTITY_TYPES, :VALID_AUTHOR_TYPES
|
||||
alias_method :"@type", :type
|
||||
private :type
|
||||
private :logo
|
||||
|
||||
# page_drop should be an instance of Jekyll::SeoTag::Drop
|
||||
def initialize(page_drop)
|
||||
|
@ -30,26 +25,17 @@ module Jekyll
|
|||
end
|
||||
|
||||
def fallback_data
|
||||
@fallback_data ||= {
|
||||
"@context" => "https://schema.org",
|
||||
{
|
||||
"@context" => "http://schema.org",
|
||||
}
|
||||
end
|
||||
|
||||
def author
|
||||
return unless page_drop.author["name"]
|
||||
|
||||
author_type = page_drop.author["type"]
|
||||
return if author_type && !VALID_AUTHOR_TYPES.include?(author_type)
|
||||
|
||||
hash = {
|
||||
"@type" => author_type || "Person",
|
||||
{
|
||||
"@type" => "Person",
|
||||
"name" => page_drop.author["name"],
|
||||
}
|
||||
|
||||
author_url = page_drop.author["url"]
|
||||
hash["url"] = author_url if author_url
|
||||
|
||||
hash
|
||||
end
|
||||
|
||||
def image
|
||||
|
@ -64,7 +50,6 @@ module Jekyll
|
|||
|
||||
def publisher
|
||||
return unless logo
|
||||
|
||||
output = {
|
||||
"@type" => "Organization",
|
||||
"logo" => {
|
||||
|
@ -77,8 +62,7 @@ module Jekyll
|
|||
end
|
||||
|
||||
def main_entity
|
||||
return unless VALID_ENTITY_TYPES.include?(type)
|
||||
|
||||
return unless %w(BlogPosting CreativeWork).include?(type)
|
||||
{
|
||||
"@type" => "WebPage",
|
||||
"@id" => page_drop.canonical_url,
|
||||
|
@ -87,15 +71,6 @@ module Jekyll
|
|||
alias_method :mainEntityOfPage, :main_entity
|
||||
private :main_entity
|
||||
|
||||
# Returns a JSON-encoded object containing the JSON-LD data.
|
||||
# Keys are sorted.
|
||||
def to_json(state = nil)
|
||||
keys.sort.each_with_object({}) do |(key, _), result|
|
||||
v = self[key]
|
||||
result[key] = v unless v.nil?
|
||||
end.to_json(state)
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
attr_reader :page_drop
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
module Jekyll
|
||||
class SeoTag
|
||||
# Mixin to share common URL-related methods between class
|
||||
|
@ -8,12 +6,11 @@ module Jekyll
|
|||
|
||||
# Determines if the given string is an absolute URL
|
||||
#
|
||||
# Returns true if an absolute URL
|
||||
# Returns false if it's a relative URL
|
||||
# Returns true if an absolute URL.
|
||||
# Retruns false if it's a relative URL
|
||||
# Returns nil if it is not a string or can't be parsed as a URL
|
||||
def absolute_url?(string)
|
||||
return unless string
|
||||
|
||||
Addressable::URI.parse(string).absolute?
|
||||
rescue Addressable::URI::InvalidURIError
|
||||
nil
|
||||
|
|
|
@ -1,10 +1,8 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
# Prevent bundler errors
|
||||
module Liquid; class Tag; end; end
|
||||
|
||||
module Jekyll
|
||||
class SeoTag < Liquid::Tag
|
||||
VERSION = "2.8.0"
|
||||
VERSION = "2.3.0".freeze
|
||||
end
|
||||
end
|
||||
|
|
|
@ -13,12 +13,11 @@
|
|||
<meta name="author" content="{{ seo_tag.author.name }}" />
|
||||
{% endif %}
|
||||
|
||||
<meta property="og:locale" content="{{ seo_tag.page_locale }}" />
|
||||
<meta property="og:locale" content="{{ seo_tag.page_lang | replace:'-','_' }}" />
|
||||
|
||||
{% if seo_tag.description %}
|
||||
<meta name="description" content="{{ seo_tag.description }}" />
|
||||
<meta property="og:description" content="{{ seo_tag.description }}" />
|
||||
<meta property="twitter:description" content="{{ seo_tag.description }}" />
|
||||
{% endif %}
|
||||
|
||||
{% if site.url %}
|
||||
|
@ -38,45 +37,31 @@
|
|||
{% if seo_tag.image.width %}
|
||||
<meta property="og:image:width" content="{{ seo_tag.image.width }}" />
|
||||
{% endif %}
|
||||
{% if seo_tag.image.alt %}
|
||||
<meta property="og:image:alt" content="{{ seo_tag.image.alt }}" />
|
||||
{% endif %}
|
||||
{% endif %}
|
||||
|
||||
{% if page.date %}
|
||||
<meta property="og:type" content="article" />
|
||||
<meta property="article:published_time" content="{{ page.date | date_to_xmlschema }}" />
|
||||
{% else %}
|
||||
<meta property="og:type" content="website" />
|
||||
{% endif %}
|
||||
|
||||
{% if paginator.previous_page %}
|
||||
<link rel="prev" href="{{ paginator.previous_page_path | absolute_url }}" />
|
||||
<link rel="prev" href="{{ paginator.previous_page_path | absolute_url }}">
|
||||
{% endif %}
|
||||
{% if paginator.next_page %}
|
||||
<link rel="next" href="{{ paginator.next_page_path | absolute_url }}" />
|
||||
{% endif %}
|
||||
|
||||
{% if seo_tag.image %}
|
||||
<meta name="twitter:card" content="{{ page.twitter.card | default: site.twitter.card | default: "summary_large_image" }}" />
|
||||
<meta property="twitter:image" content="{{ seo_tag.image.path }}" />
|
||||
{% else %}
|
||||
<meta name="twitter:card" content="summary" />
|
||||
{% endif %}
|
||||
|
||||
{% if seo_tag.image.alt %}
|
||||
<meta name="twitter:image:alt" content="{{ seo_tag.image.alt }}" />
|
||||
{% endif %}
|
||||
|
||||
{% if seo_tag.page_title %}
|
||||
<meta property="twitter:title" content="{{ seo_tag.page_title }}" />
|
||||
<link rel="next" href="{{ paginator.next_page_path | absolute_url }}">
|
||||
{% endif %}
|
||||
|
||||
{% if site.twitter %}
|
||||
<meta name="twitter:site" content="@{{ site.twitter.username | remove:'@' }}" />
|
||||
{% if seo_tag.image %}
|
||||
<meta name="twitter:card" content="summary_large_image" />
|
||||
{% else %}
|
||||
<meta name="twitter:card" content="summary" />
|
||||
{% endif %}
|
||||
|
||||
<meta name="twitter:site" content="@{{ site.twitter.username | replace:"@","" }}" />
|
||||
|
||||
{% if seo_tag.author.twitter %}
|
||||
<meta name="twitter:creator" content="@{{ seo_tag.author.twitter | remove:'@' }}" />
|
||||
<meta name="twitter:creator" content="@{{ seo_tag.author.twitter }}" />
|
||||
{% endif %}
|
||||
{% endif %}
|
||||
|
||||
|
@ -96,27 +81,19 @@
|
|||
|
||||
{% if site.webmaster_verifications %}
|
||||
{% if site.webmaster_verifications.google %}
|
||||
<meta name="google-site-verification" content="{{ site.webmaster_verifications.google }}" />
|
||||
<meta name="google-site-verification" content="{{ site.webmaster_verifications.google }}">
|
||||
{% endif %}
|
||||
|
||||
{% if site.webmaster_verifications.bing %}
|
||||
<meta name="msvalidate.01" content="{{ site.webmaster_verifications.bing }}" />
|
||||
<meta name="msvalidate.01" content="{{ site.webmaster_verifications.bing }}">
|
||||
{% endif %}
|
||||
|
||||
{% if site.webmaster_verifications.alexa %}
|
||||
<meta name="alexaVerifyID" content="{{ site.webmaster_verifications.alexa }}" />
|
||||
<meta name="alexaVerifyID" content="{{ site.webmaster_verifications.alexa }}">
|
||||
{% endif %}
|
||||
|
||||
{% if site.webmaster_verifications.yandex %}
|
||||
<meta name="yandex-verification" content="{{ site.webmaster_verifications.yandex }}" />
|
||||
{% endif %}
|
||||
|
||||
{% if site.webmaster_verifications.baidu %}
|
||||
<meta name="baidu-site-verification" content="{{ site.webmaster_verifications.baidu }}" />
|
||||
{% endif %}
|
||||
|
||||
{% if site.webmaster_verifications.facebook %}
|
||||
<meta name="facebook-domain-verification" content="{{ site.webmaster_verifications.facebook }}" />
|
||||
<meta name="yandex-verification" content="{{ site.webmaster_verifications.yandex }}">
|
||||
{% endif %}
|
||||
{% elsif site.google_site_verification %}
|
||||
<meta name="google-site-verification" content="{{ site.google_site_verification }}" />
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
RSpec.describe Jekyll::SeoTag::AuthorDrop do
|
||||
let(:data) { {} }
|
||||
let(:config) { { "author" => "site_author" } }
|
||||
|
@ -50,7 +48,7 @@ RSpec.describe Jekyll::SeoTag::AuthorDrop do
|
|||
end
|
||||
end
|
||||
|
||||
[:with, :without].each do |site_data_type|
|
||||
%i[with without].each do |site_data_type|
|
||||
context "#{site_data_type} site.author data" do
|
||||
let(:data) do
|
||||
if site_data_type == :with
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
RSpec.describe Jekyll::SeoTag::Drop do
|
||||
let(:config) { { "title" => "site title" } }
|
||||
let(:page_meta) { { "title" => "page title" } }
|
||||
|
@ -84,17 +82,6 @@ RSpec.describe Jekyll::SeoTag::Drop do
|
|||
end
|
||||
end
|
||||
|
||||
context "with a site tagline but no page title" do
|
||||
let(:page) { make_page }
|
||||
let(:config) do
|
||||
{ "title" => "site title", "description" => "site description", "tagline" => "site tagline" }
|
||||
end
|
||||
|
||||
it "builds the title" do
|
||||
expect(subject.title).to eql("site title | site tagline")
|
||||
end
|
||||
end
|
||||
|
||||
context "with just a page title" do
|
||||
let(:site) { make_site }
|
||||
|
||||
|
@ -233,24 +220,6 @@ RSpec.describe Jekyll::SeoTag::Drop do
|
|||
expect(subject.description).to be_nil
|
||||
end
|
||||
end
|
||||
|
||||
context "truncation" do
|
||||
context "without seo_description_max_words" do
|
||||
let(:page_meta) { { "description" => "word " * 150 } }
|
||||
|
||||
it "truncates the description to the first 200 words" do
|
||||
expect(subject.description).to eql(("word " * 100).chop.concat("…"))
|
||||
end
|
||||
end
|
||||
|
||||
context "with an explicit seo_description_max_words property" do
|
||||
let(:page_meta) { { "description" => "For a long time, I went to bed early", "seo_description_max_words" => 6 } }
|
||||
|
||||
it "truncates the description to the configured words count" do
|
||||
expect(subject.description).to eql("For a long time, I went…")
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context "author" do
|
||||
|
@ -349,7 +318,7 @@ RSpec.describe Jekyll::SeoTag::Drop do
|
|||
end
|
||||
|
||||
context "the about page" do
|
||||
let(:page) { make_page("permalink" => "/about/") }
|
||||
let(:page) { make_page({ "permalink" => "/about/" }) }
|
||||
|
||||
it "is a website" do
|
||||
expect(subject.type).to eql("WebSite")
|
||||
|
@ -433,7 +402,7 @@ RSpec.describe Jekyll::SeoTag::Drop do
|
|||
end
|
||||
|
||||
context "image" do
|
||||
let(:image) { "/foo.png" }
|
||||
let(:image) { "foo.png" }
|
||||
let(:page_meta) { { "image" => image } }
|
||||
|
||||
it "returns a Drop" do
|
||||
|
@ -511,27 +480,6 @@ RSpec.describe Jekyll::SeoTag::Drop do
|
|||
end
|
||||
end
|
||||
|
||||
context "pagination" do
|
||||
let(:context) do
|
||||
make_context(
|
||||
{ :page => page, :site => site },
|
||||
"paginator" => { "page" => 2, "total_pages" => 10 }
|
||||
)
|
||||
end
|
||||
|
||||
it "render default pagination title" do
|
||||
expect(subject.send(:page_number)).to eq("Page 2 of 10 for ")
|
||||
end
|
||||
|
||||
context "render custom pagination title" do
|
||||
let(:config) { { "seo_paginator_message" => "%<current>s of %<total>s" } }
|
||||
|
||||
it "renders the correct page number" do
|
||||
expect(subject.send(:page_number)).to eq("2 of 10")
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
it "exposes the JSON-LD drop" do
|
||||
expect(subject.json_ld).to be_a(Jekyll::SeoTag::JSONLDDrop)
|
||||
end
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
RSpec.describe Jekyll::SeoTag::Filters do
|
||||
let(:page) { make_page }
|
||||
let(:site) { make_site }
|
||||
|
|
|
@ -1,9 +1,7 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
RSpec.describe Jekyll::SeoTag::ImageDrop do
|
||||
let(:config) { { "title" => "site title" } }
|
||||
let(:image) { nil }
|
||||
let(:page_meta) { { "image" => image, "dir" => "foo" } }
|
||||
let(:page_meta) { { "image" => image } }
|
||||
let(:page) { make_page(page_meta) }
|
||||
let(:site) { make_site(config) }
|
||||
let(:context) { make_context(:page => page, :site => site) }
|
||||
|
@ -14,34 +12,8 @@ RSpec.describe Jekyll::SeoTag::ImageDrop do
|
|||
Jekyll.logger.log_level = :error
|
||||
end
|
||||
|
||||
context "with a post object" do
|
||||
context "with image as a string" do
|
||||
let(:image) { "image.png" }
|
||||
let(:page_meta) { { "image" => image, "date" => "2017-01-01" } }
|
||||
let(:page) { make_post(page_meta) }
|
||||
|
||||
it "returns the image url relative to the post directory" do
|
||||
expect(subject["path"]).to eql("/2017/01/01/image.png")
|
||||
end
|
||||
end
|
||||
|
||||
context "with image as a relative path" do
|
||||
let(:image) { "image.png" }
|
||||
|
||||
it "returns the image with the page dir prepended" do
|
||||
expect(subject["path"]).to eql("/foo/image.png")
|
||||
end
|
||||
|
||||
context "with site.url" do
|
||||
let(:config) { { "url" => "http://example.com" } }
|
||||
|
||||
it "makes the path absolute" do
|
||||
expect(subject["path"]).to eql("http://example.com/foo/image.png")
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context "with image as an absolute path" do
|
||||
let(:image) { "/image.png" }
|
||||
|
||||
it "returns the image" do
|
||||
expect(subject["path"]).to eql("/image.png")
|
||||
|
@ -56,7 +28,7 @@ RSpec.describe Jekyll::SeoTag::ImageDrop do
|
|||
end
|
||||
|
||||
context "with a URL-escaped path" do
|
||||
let(:image) { "/some image.png" }
|
||||
let(:image) { "some image.png" }
|
||||
|
||||
it "URL-escapes the image" do
|
||||
expect(subject["path"]).to eql("/some%20image.png")
|
||||
|
@ -65,8 +37,8 @@ RSpec.describe Jekyll::SeoTag::ImageDrop do
|
|||
end
|
||||
|
||||
context "with image as a hash" do
|
||||
context "with an absolute path" do
|
||||
let(:image) { { "path" => "/image.png" } }
|
||||
context "with a path" do
|
||||
let(:image) { { "path" => "image.png" } }
|
||||
|
||||
it "returns the image" do
|
||||
expect(subject["path"]).to eql("/image.png")
|
||||
|
@ -74,7 +46,7 @@ RSpec.describe Jekyll::SeoTag::ImageDrop do
|
|||
end
|
||||
|
||||
context "with facebook" do
|
||||
let(:image) { { "facebook" => "/image.png" } }
|
||||
let(:image) { { "facebook" => "image.png" } }
|
||||
|
||||
it "returns the image" do
|
||||
expect(subject["path"]).to eql("/image.png")
|
||||
|
@ -82,7 +54,7 @@ RSpec.describe Jekyll::SeoTag::ImageDrop do
|
|||
end
|
||||
|
||||
context "with twitter" do
|
||||
let(:image) { { "twitter" => "/image.png" } }
|
||||
let(:image) { { "twitter" => "image.png" } }
|
||||
|
||||
it "returns the image" do
|
||||
expect(subject["path"]).to eql("/image.png")
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
RSpec.describe Jekyll::SeoTag::JSONLDDrop do
|
||||
let(:author) { "author" }
|
||||
let(:image) { "image" }
|
||||
|
@ -35,7 +33,7 @@ RSpec.describe Jekyll::SeoTag::JSONLDDrop do
|
|||
|
||||
it "returns the context" do
|
||||
expect(subject).to have_key("@context")
|
||||
expect(subject["@context"]).to eql("https://schema.org")
|
||||
expect(subject["@context"]).to eql("http://schema.org")
|
||||
end
|
||||
|
||||
it "returns the type" do
|
||||
|
@ -69,52 +67,14 @@ RSpec.describe Jekyll::SeoTag::JSONLDDrop do
|
|||
expect(subject["author"]).to have_key("name")
|
||||
expect(subject["author"]["name"]).to be_a(String)
|
||||
expect(subject["author"]["name"]).to eql("author")
|
||||
expect(subject["author"]["url"]).to be nil
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context "when type Organization" do
|
||||
let(:author) { { "name" => "organization", "type" => "Organization" } }
|
||||
|
||||
it "returns the author with type" do
|
||||
expect(subject).to have_key("author")
|
||||
expect(subject["author"]).to be_a(Hash)
|
||||
expect(subject["author"]).to have_key("@type")
|
||||
expect(subject["author"]["@type"]).to eql("Organization")
|
||||
expect(subject["author"]).to have_key("name")
|
||||
expect(subject["author"]["name"]).to be_a(String)
|
||||
expect(subject["author"]["name"]).to eql("organization")
|
||||
end
|
||||
end
|
||||
|
||||
context "when invalid type" do
|
||||
let(:author) { { "name" => "organization", "type" => "Invalid" } }
|
||||
|
||||
it "returns the author with type" do
|
||||
expect(subject).to have_key("author")
|
||||
expect(subject["author"]).to be nil
|
||||
end
|
||||
end
|
||||
|
||||
context "when url is present" do
|
||||
let(:author) { { "name" => "author", "url" => "https://example.com" } }
|
||||
|
||||
it "returns the author with url" do
|
||||
expect(subject).to have_key("author")
|
||||
expect(subject["author"]).to be_a(Hash)
|
||||
expect(subject["author"]).to have_key("url")
|
||||
expect(subject["author"]["url"]).to eql("https://example.com")
|
||||
expect(subject["author"]).to have_key("name")
|
||||
expect(subject["author"]["name"]).to be_a(String)
|
||||
expect(subject["author"]["name"]).to eql("author")
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context "image" do
|
||||
context "with image as an absolute path" do
|
||||
let(:image) { "/image" }
|
||||
context "with image as a string" do
|
||||
let(:image) { "image" }
|
||||
|
||||
it "returns the image as a string" do
|
||||
expect(subject).to have_key("image")
|
||||
|
@ -124,17 +84,17 @@ RSpec.describe Jekyll::SeoTag::JSONLDDrop do
|
|||
end
|
||||
|
||||
context "with image as a hash" do
|
||||
let(:image) { { "path" => "/image", "height" => 5, "width" => 10 } }
|
||||
let(:image) { { "path" => "image", "height" => 5, "width" => 10 } }
|
||||
|
||||
it "returns the image as a hash" do
|
||||
expect(subject).to have_key("image")
|
||||
expect(subject["image"]).to be_a(Hash)
|
||||
expect(subject["image"]).to eql(
|
||||
expect(subject["image"]).to eql({
|
||||
"@type" => "imageObject",
|
||||
"url" => "/image",
|
||||
"height" => 5,
|
||||
"width" => 10
|
||||
)
|
||||
"width" => 10,
|
||||
})
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -190,12 +150,4 @@ RSpec.describe Jekyll::SeoTag::JSONLDDrop do
|
|||
expect(subject).to have_key("url")
|
||||
expect(subject["url"]).to eql("/page.html")
|
||||
end
|
||||
|
||||
context "with null values" do
|
||||
let(:metadata) { {} }
|
||||
|
||||
it "does not return null values as json" do
|
||||
expect(subject.to_json).to_not match(%r!:null!)
|
||||
end
|
||||
end
|
||||
end
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
RSpec.describe Jekyll::SeoTag do
|
||||
let(:page) { make_page }
|
||||
let(:site) { make_site }
|
||||
|
@ -8,7 +6,7 @@ RSpec.describe Jekyll::SeoTag do
|
|||
let(:tag) { "seo" }
|
||||
let(:text) { "" }
|
||||
let(:output) { Liquid::Template.parse("{% #{tag} #{text} %}").render!(context, {}) }
|
||||
let(:json) { output.match(%r!<script type="application/ld\+json">(.*)</script>!m)[1] }
|
||||
let(:json) { output.match(%r!<script type=\"application/ld\+json\">(.*)</script>!m)[1] }
|
||||
let(:json_data) { JSON.parse(json) }
|
||||
let(:paginator) { { "previous_page" => true, "previous_page_path" => "foo", "next_page" => true, "next_page_path" => "bar" } }
|
||||
|
||||
|
@ -26,12 +24,7 @@ RSpec.describe Jekyll::SeoTag do
|
|||
end
|
||||
|
||||
it "outputs meta generator" do
|
||||
version = Jekyll::VERSION
|
||||
expect(output).to match(%r!Jekyll v#{version}!i)
|
||||
end
|
||||
|
||||
it "outputs JSON sorted by key" do
|
||||
expect(json.strip).to eql('{"@context":"https://schema.org","@type":"WebPage","url":"/page.html"}')
|
||||
expect(output).to match(%r!Jekyll v#{Jekyll::VERSION}!i)
|
||||
end
|
||||
|
||||
it "outputs valid HTML" do
|
||||
|
@ -124,7 +117,6 @@ RSpec.describe Jekyll::SeoTag do
|
|||
it "uses the page description" do
|
||||
expect(output).to match(%r!<meta name="description" content="foo" />!)
|
||||
expect(output).to match(%r!<meta property="og:description" content="foo" />!)
|
||||
expect(output).to match(%r!<meta property="twitter:description" content="foo" />!)
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -134,7 +126,6 @@ RSpec.describe Jekyll::SeoTag do
|
|||
it "uses the page excerpt when no page description exists" do
|
||||
expect(output).to match(%r!<meta name="description" content="foo" />!)
|
||||
expect(output).to match(%r!<meta property="og:description" content="foo" />!)
|
||||
expect(output).to match(%r!<meta property="twitter:description" content="foo" />!)
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -144,7 +135,6 @@ RSpec.describe Jekyll::SeoTag do
|
|||
it "uses the site description when no page description nor excerpt exist" do
|
||||
expect(output).to match(%r!<meta name="description" content="foo" />!)
|
||||
expect(output).to match(%r!<meta property="og:description" content="foo" />!)
|
||||
expect(output).to match(%r!<meta property="twitter:description" content="foo" />!)
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -307,16 +297,16 @@ RSpec.describe Jekyll::SeoTag do
|
|||
|
||||
it "minifies the output" do
|
||||
version = Jekyll::SeoTag::VERSION
|
||||
expected = <<~HTML
|
||||
<!-- Begin Jekyll SEO tag v#{version} -->
|
||||
<title>Foo</title>
|
||||
<meta name="generator" content="Jekyll v#{Jekyll::VERSION}" />
|
||||
<meta property="og:title" content="Foo" />
|
||||
<meta property="og:locale" content="en_US" />
|
||||
<link rel="canonical" href="http://example.invalid/page.html" />
|
||||
<meta property="og:url" content="http://example.invalid/page.html" />
|
||||
<meta property="og:site_name" content="Foo" />
|
||||
HTML
|
||||
expected = <<-EOS
|
||||
<!-- Begin Jekyll SEO tag v#{version} -->
|
||||
<title>Foo</title>
|
||||
<meta name="generator" content="Jekyll v#{Jekyll::VERSION}" />
|
||||
<meta property="og:title" content="Foo" />
|
||||
<meta property="og:locale" content="en_US" />
|
||||
<link rel="canonical" href="http://example.invalid/page.html" />
|
||||
<meta property="og:url" content="http://example.invalid/page.html" />
|
||||
<meta property="og:site_name" content="Foo" />
|
||||
EOS
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
end
|
||||
|
@ -346,27 +336,6 @@ RSpec.describe Jekyll::SeoTag do
|
|||
it "minifies JSON-LD" do
|
||||
expect(output).to_not match(%r!{.*?\s.*?}!)
|
||||
end
|
||||
|
||||
it "removes null values from JSON-LD" do
|
||||
expect(output).to_not match(%r!:null!)
|
||||
end
|
||||
|
||||
context "description" do
|
||||
context "with page.seo_description_max_words" do
|
||||
let(:meta) do
|
||||
{
|
||||
"title" => "post",
|
||||
"description" => "For a long time, I went to bed early",
|
||||
"image" => "/img.png",
|
||||
"seo_description_max_words" => 6,
|
||||
}
|
||||
end
|
||||
|
||||
it "truncates the description" do
|
||||
expect(json_data["description"]).to eql("For a long time, I went…")
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -449,34 +418,11 @@ RSpec.describe Jekyll::SeoTag do
|
|||
end
|
||||
|
||||
context "with page.image" do
|
||||
context "without *.twitter.card" do
|
||||
let(:page) { make_page("image" => "/img/foo.png") }
|
||||
let(:page) { make_page("image" => "/img/foo.png") }
|
||||
|
||||
it "outputs summary card with large image" do
|
||||
expected = %r!<meta name="twitter:card" content="summary_large_image" />!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
end
|
||||
|
||||
context "with page.twitter.card" do
|
||||
let(:page_twitter) { { "card" => "summary" } }
|
||||
let(:page) { make_page("image" => "/img/foo.png", "twitter" => page_twitter) }
|
||||
|
||||
it "outputs summary card with small image" do
|
||||
expected = %r!<meta name="twitter:card" content="summary" />!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
end
|
||||
|
||||
context "with site.twitter.card" do
|
||||
let(:site_twitter) { { "username" => "jekyllrb", "card" => "summary" } }
|
||||
let(:site) { make_site("twitter" => site_twitter) }
|
||||
let(:page) { make_page("image" => "/img/foo.png") }
|
||||
|
||||
it "outputs summary card with small image" do
|
||||
expected = %r!<meta name="twitter:card" content="summary" />!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
it "outputs summary card with large image" do
|
||||
expected = %r!<meta name="twitter:card" content="summary_large_image" />!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -629,8 +575,8 @@ RSpec.describe Jekyll::SeoTag do
|
|||
let(:context) { make_context({}, "paginator" => paginator) }
|
||||
|
||||
it "outputs pagination links" do
|
||||
expect(output).to match(%r!<link rel="prev" href="/foo" />!)
|
||||
expect(output).to match(%r!<link rel="next" href="/bar" />!)
|
||||
expect(output).to match(%r!<link rel="prev" href="/foo">!)
|
||||
expect(output).to match(%r!<link rel="next" href="/bar">!)
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -638,38 +584,32 @@ RSpec.describe Jekyll::SeoTag do
|
|||
context "with site.webmaster_verifications" do
|
||||
let(:site_verifications) do
|
||||
{
|
||||
"google" => "foo",
|
||||
"bing" => "bar",
|
||||
"alexa" => "baz",
|
||||
"yandex" => "bat",
|
||||
"facebook" => "bas",
|
||||
"google" => "foo",
|
||||
"bing" => "bar",
|
||||
"alexa" => "baz",
|
||||
"yandex" => "bat",
|
||||
}
|
||||
end
|
||||
|
||||
let(:site) { make_site("webmaster_verifications" => site_verifications) }
|
||||
|
||||
it "outputs google verification meta" do
|
||||
expected = %r!<meta name="google-site-verification" content="foo" />!
|
||||
expected = %r!<meta name="google-site-verification" content="foo">!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
|
||||
it "outputs bing verification meta" do
|
||||
expected = %r!<meta name="msvalidate.01" content="bar" />!
|
||||
expected = %r!<meta name="msvalidate.01" content="bar">!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
|
||||
it "outputs alexa verification meta" do
|
||||
expected = %r!<meta name="alexaVerifyID" content="baz" />!
|
||||
expected = %r!<meta name="alexaVerifyID" content="baz">!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
|
||||
it "outputs yandex verification meta" do
|
||||
expected = %r!<meta name="yandex-verification" content="bat" />!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
|
||||
it "outputs facebook verification meta" do
|
||||
expected = %r!<meta name="facebook-domain-verification" content="bas" />!
|
||||
expected = %r!<meta name="yandex-verification" content="bat">!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
end
|
||||
|
@ -691,10 +631,10 @@ RSpec.describe Jekyll::SeoTag do
|
|||
end
|
||||
|
||||
context "with site.lang" do
|
||||
let(:site) { make_site("lang" => "de_DE") }
|
||||
let(:site) { make_site("lang" => "en_US") }
|
||||
|
||||
it "uses site.lang if page.lang is not present" do
|
||||
expected = %r!<meta property="og:locale" content="de_DE" />!
|
||||
expected = %r!<meta property="og:locale" content="en_US" />!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
|
||||
|
@ -709,10 +649,10 @@ RSpec.describe Jekyll::SeoTag do
|
|||
end
|
||||
|
||||
context "with site.lang hyphenated" do
|
||||
let(:site) { make_site("lang" => "en-AU") }
|
||||
let(:site) { make_site("lang" => "en-US") }
|
||||
|
||||
it "coerces hyphen to underscore" do
|
||||
expected = %r!<meta property="og:locale" content="en_AU" />!
|
||||
expected = %r!<meta property="og:locale" content="en_US" />!
|
||||
expect(output).to match(expected)
|
||||
end
|
||||
end
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
RSpec.describe Jekyll::SeoTag do
|
||||
let(:config) { { "title" => "site title" } }
|
||||
let(:page_meta) { {} }
|
||||
|
|
|
@ -1,5 +1,3 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
$LOAD_PATH.unshift File.expand_path("../lib", __dir__)
|
||||
require "jekyll"
|
||||
require "jekyll-seo-tag"
|
||||
|
@ -31,7 +29,7 @@ CONFIG_DEFAULTS = {
|
|||
}.freeze
|
||||
|
||||
def make_page(options = {})
|
||||
page = Jekyll::Page.new site, CONFIG_DEFAULTS["source"], options.delete("dir") || "", "page.md"
|
||||
page = Jekyll::Page.new site, CONFIG_DEFAULTS["source"], "", "page.md"
|
||||
page.data = options
|
||||
page
|
||||
end
|
||||
|
|
Loading…
Reference in New Issue