Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

The samples are all completely independent and self-contained.

オープン
#58 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
35/100
issue の種類
ドキュメント
明瞭さ
説明が足りない
活発さ
停滞
技術スタック
php
領域
documentation

調査の方向性

README の「Using the Sample Code」セクションと、そこで参照されている Samples/Payments/Payments/AuthorizationWithCaptureSale.php ファイルから始めてください。「independent」と「self-contained」が何を意味するものとして意図されているのかを確認し、そのうえで、文言を記載されている依存関係に合わせて更新し、リンクと例が引き続き正確であることを検証してください。

索引モデルが issue の本文から書いたものです。

説明

Under Readme section "Using the Sample Code" - https://github.com/CyberSource/cybersource-rest-samples-php#using-the-sample-code - it states "The samples are all completely independent and self-contained.", but after reviewing the code, this doesn't appear to be true.

Example:
https://github.com/CyberSource/cybersource-rest-samples-php/blob/master/Samples/Payments/Payments/AuthorizationWithCaptureSale.php

This file has code:

require_once __DIR__ . DIRECTORY_SEPARATOR . '../../../Resources/ExternalConfiguration.php';

which is another file in the repo. Maybe I'm reading the blurb wrong, but that file to me is a sample, and if it's self-contained it shouldn't require other files.

Anyway, even if I consider "self-contained" samples to mean "the repo is self-contained as a run app", then that's not exactly true either. The above line of code requires vendor through composer that's compatible with PHP 5.6:

require_once __DIR__ . DIRECTORY_SEPARATOR . '../../../vendor/autoload.php';

This is required for at least one line in that AuthorizationWithCaptureSale.php sample file's code, like:

$api_instance = new CyberSource\Api\PaymentsApi($api_client);

I'm working on a Mac (Ventura) that doesn't have PHP 5.6 or a compatible version of composer installed, so I will have to do a deep-dive to figure that out for a "self-contained" sample file.

Anyway, I can try to get the whole repo with vendor code running, but to me the blurb is not accurate, as the sample files are not self-contained. I'm curious exactly what's meant by "self-contained" or "independent".

主要言語
JavaScript
スター
43
フォーク
66
平均マージ
1日 8時間
マージ済み PR(30日)
1

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

CyberSource/cybersource-rest-samples-node のほかの issue

CyberSource/cybersource-rest-samples-node の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。