[Olein_jp]

エラーらしいエラーは出てませんでしたね…

もう少しもがいてみます！ありがとうございました！！

♥ 0

いいねをした人: 居ません

[Olein_jp]

先ほど頂いたソースコードを適宜変更してテストしてみたのですが、デモ環境にアップデートは走っているので大丈夫なのかもしれないのですが、Releasesからパッケージが削除されていないので、それがGitHubを参照しているのか自前サーバーを参照しているのか確認できない…という状況です。

webhook.phpの最下部をコメントアウトしてみてはいるのですが…。

♥ 0

いいねをした人: 居ません

[Olein_jp]

ありがとうございます。

/**
 * There is a case that comes back to GitHub's zip url.
 * In that case it returns false because it is illegal.
 *
 * @param string $url
 * @return string|false
 */
add_filter(
    'inc2734_github_theme_updater_zip_url',
    function( $url ) {
        if ( 0 !== strpos( $url, 'https://snow-monkey.2inc.org/' ) ) {
            return false;
        }
        return $url;
    }
);

この部分のURLなんですが、ここではzipファイルが格納される予定（config.phpのINSTALLATION_URLと同じになる）のURLでしょうか？それともドメイン（サブドメインレベル）でもOKでしょうか？

♥ 0

いいねをした人: 居ません

[Olein_jp]

なるほど！

ちょっとこのプラグインを試してみることにします！

♥ 0

いいねをした人: 居ません

[Olein_jp]

kuro81さん、コメントありがとうございます。

ウィジェットが数年後にはカスタムブロックに切り替わって行く予定なので、注力するべきかどうかは北島さん判断で良いと思うんですが、需要としてはありそうな気がしました。（というか、自分が欲しいだけ？w

カスタムブロックのカスタムHTMLベースで実装とかでもいいかなーと思うんですが、そうすると投稿毎にブロック追加しないといけないので、であれば再利用ブロックでなんとかならなくもないなーと。

（勝手にですが）意外にありな気がしてます。

♥ 0

いいねをした人: 居ません

[Olein_jp]

すいません。僕、北島さんじゃないです…。

なんかごめんなさい…解決されたならよかったです…

♥ 2

いいねをした人: 居ません

[Olein_jp]

<script>〜</script>タグが抜けてしまっているので添付画像のようにソースが表に見えてしまっている状況のようですね。

Adsense側から提供されたソースコード（下記サンプルのようなもの）をそのままコピペしてみてください。

<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
		<script>
            (adsbygoogle = window.adsbygoogle || []).push({
                google_ad_client: "ca-pub-**************",
                enable_page_level_ads: true
            });
		</script>

♥ 0

いいねをした人: 居ません

[Olein_jp]

<?php
add_action( ‘wp_enqueue_scripts’, function() {
wp_enqueue_style(
get_stylesheet(),
get_stylesheet_uri(),
[ get_template() ]
);
} );

add_action(
	'wp_head',
	function() {
		?>
		ここにアドセンスのコード
		<?php
	}
);

こういう感じで大丈夫かと思います！

♥ 0

いいねをした人: 居ません

[Olein_jp]

ご確認いただきありがとうございます。

方法としては僕も子テーマのフックで追加すれば良いかなと考えていたのですが、Snow Monkey ガイドラインがあれば、そちらの方法でやるべきかと思い、質問させていただきました。

ありがとうございます！

♥ 0

いいねをした人: 居ません

[Olein_jp]

あ、すいません。。。。

キャッシュを全てクリアにしたら、正常に動作を確認できました…汗

♥ 0

いいねをした人: 居ません

[Olein_jp]

before_scriptの前で上記処理を記述したらデプロイも成功しました！

しかし、現状としては、なんとか北島先生の力を借りてデプロイがやっとできた…という状況ですので、もっと理解を深めないといけないなぁ…というところで落ち着きました。

アドバイスいただけて大変助かりました！ありがとうございました！！

♥ 0

いいねをした人: 居ません

[Olein_jp]

アドバイスいただいたように、

on:
    tags: true
    php:
      - 7.1
    repo: Olein-jp/oleinpress-weblog

を

on:
    tags: true
    repo: Olein-jp/oleinpress-weblog

にしてプッシュしてみました。そしたら、デプロイまで走ったみたいです！ですが、

rimraf: not found

と言われるんですけど、beforeかどこかでrimrafを使えるように書いておく必要があるんですかね・・・？

♥ 0

いいねをした人: 居ません

[Olein_jp]

before_deployの前に

before_script: composer require atoum/atoum
script: vendor/bin/atoum

を入れたらテストは無事にパスしました！！！ありがとうございます！！

あと、ついでに質問させていただきたいのですが、GitHubのReleasesページにタグ付けされたバージョンと共にZipファイルが掲載されていくのですが、ファイル名にバージョン名がついてしまう（oleinpress-weblog-0.0.4.zipとか）原因って何かご存知だったりしますでしょうか？

Releases · Olein-jp/oleinpress-weblog

Contribute to Olein-jp/oleinpress-weblog development by creating an account on GitHub.

♥ 0

いいねをした人: 居ません

[Olein_jp]

そうなんですよ。

僕もこの$ phpunitがなぜ実行されているのかが全然わかんないんです…

before_deployで実行しているのも、npm run build npm run zip export RELEASE_FILE=$(ls oleinpress-weblog.zip)だけですし、それぞれの処理も記事内のものとほとんど変わりないし…

なかなか手強いですな…

♥ 0

いいねをした人: 居ません

[Olein_jp]

無事にexclude設定することができました。

それと同時になぜdistとかresourceとかディレクトリを分けるのかの意味もなんとなくつかむことができました。ありがとうございました！！！

♥ 0

いいねをした人: 居ません

[投稿者]

投稿