danger / danger/danger-js

REPL does not include any current Dangerfile context

Open
#237 4 comments 0 reactions 0 assignees View on GitHub
bug enhancement You Can Do This
Dominant language
TypeScript
Stars
5.5k
Forks
381
Avg merge
2h 8m
Merged PRs (30d)
6

Description

When using something like: `yarn danger -- pr https://github.com/artsy/emission/pull/499 --repl`

It currently doesn't matter what you've set up in your Dangerfile, it won't pass on the local context in the form of `const`s, functions et al. This threw me off, had to C&P a bunch of work before I could dig in.

Ideally the REPL's context would include all of the Dangerfile's current context. Maybe we can include some text beforehand saying how to use the repl too.

Contributor guide

Open the contributing guide

Research direction

Start by running the documented command, `yarn danger -- pr https://github.com/artsy/emission/pull/499 --repl`, and trace how the REPL loads the Dangerfile. Check which current Dangerfile context, including consts and functions, is available in the REPL. Done means the REPL preserves that context and, if added, explains how to use it.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
cli, developer-experience
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.