Skip to content

Commit

Permalink
Remove is_fork
Browse files Browse the repository at this point in the history
  • Loading branch information
frankharkins committed Dec 4, 2024
1 parent e673c5c commit 83b5eed
Showing 1 changed file with 0 additions and 24 deletions.
24 changes: 0 additions & 24 deletions scripts/nb-tester/qiskit_docs_notebook_tester/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -164,22 +164,6 @@ def notebooks_to_execute(self) -> Iterator[Path]:
)
continue

if self.args.is_fork:
print(
"⛔️ We can't run notebook tests on pull requests from forks "
"because of how GitHub Secrets work."
"\n\n"
"If you have write access to Qiskit/documentation, push to a new "
"branch there and make your pull request from that branch instead."
"\n\n"
"If you don't have write access, you should locally test out the notebooks you're modifying "
"by using the instructions in "
"https://github.com/Qiskit/documentation#execute-notebooks. "
"When this PR is approved, a maintainer will merge it to a new "
"branch in Qiskit/documentation, then make a PR from that branch "
"into main so it can pass CI.\n",
)
sys.exit(1)
yield path

def should_patch(self, path: Path) -> bool:
Expand Down Expand Up @@ -242,14 +226,6 @@ def get_args() -> argparse.Namespace:
"--config-path",
help="Path to a TOML file containing the globs for detecting and sorting notebooks",
)
parser.add_argument(
"--is-fork",
action="store_true",
help=(
"Set to true when running on forks in CI, since authentication cannot work there. "
"The program will fail with a helpful message if any of the notebooks cannot be executed."
),
)
parser.add_argument(
"--provider",
action="store",
Expand Down

0 comments on commit 83b5eed

Please sign in to comment.