Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate data.js using DangerJS #999

Open
davidak opened this issue Oct 13, 2020 · 5 comments
Open

Validate data.js using DangerJS #999

davidak opened this issue Oct 13, 2020 · 5 comments
Labels

Comments

@davidak
Copy link
Collaborator

davidak commented Oct 13, 2020

https://github.com/wesbos/awesome-uses/pull/998/checks

Run npx danger ci
You need to add GITHUB_TOKEN to your Danger action in the workflow:
  
    - name: Danger JS
      uses: danger/[email protected]
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
    
The DANGER_GITHUB_API_TOKEN/DANGER_BITBUCKETSERVER_HOST/DANGER_GITLAB_API_TOKEN environmental variable is missing
Without an api token, danger will be unable to comment on a PR
Error:  Error: Cannot use authenticated API requests.
    at Object.getPlatformForEnv (/home/runner/work/awesome-uses/awesome-uses/node_modules/danger/distribution/platforms/platform.js:76:11)
    at Object.<anonymous> (/home/runner/work/awesome-uses/awesome-uses/node_modules/danger/distribution/commands/ci/runner.js:81:57)
    at step (/home/runner/work/awesome-uses/awesome-uses/node_modules/danger/distribution/commands/ci/runner.js:32:23)
    at Object.next (/home/runner/work/awesome-uses/awesome-uses/node_modules/danger/distribution/commands/ci/runner.js:13:53)
    at fulfilled (/home/runner/work/awesome-uses/awesome-uses/node_modules/danger/distribution/commands/ci/runner.js:4:58)
Error: Process completed with exit code 1.
@davidak davidak added the bug label Oct 13, 2020
@Joshua-rose
Copy link
Contributor

Received the same with #1000

@jorgearuv
Copy link
Contributor

I'm getting the same in #1004

@HugoDF
Copy link
Collaborator

HugoDF commented Oct 16, 2020

Yeah this is one for @wesbos he needs to give dangerJS the right permissions

@rohit-gohri
Copy link
Contributor

I mentioned this in the original issue #478 (comment) for automating checks. But forgot to mention in the PR (#796) that a secret DANGER_GITHUB_API_TOKEN will need to be added to the project and now it's merged without access to the repo 🙈.

@itsmanojb
Copy link
Contributor

Still failing.

Screenshot from 2020-10-28 02-25-42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants