Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

deivid-rodriguez
Copy link
Contributor

Because no testdata files are actually copied to the image, we fail to read the event file and fallback to fetching the information from GitHub API.

Howver, that's not actually working because the GITHUB_API_URL environment is missing.

To fix it, we could pass the GITHUB_API_URL. However, I think actually making the testdata file available to the container mimics better what happens in real life.

This is an alternative to #1582.

docs/development.md Outdated Show resolved Hide resolved
Because no testdata files are actually copied to the image, we fail to
read the event file and fallback to fetching the information from GitHub
API.

Howver, that's not actually working because the `GITHUB_API_URL`
environment is missing.

To fix it, we could pass the `GITHUB_API_URL`. However, I think actually
making the testdata file available to the container mimics better what
happens in real life.

Signed-off-by: David Rodríguez <2887858+deivid-rodriguez@users.noreply.github.com>
Copy link
Member

@spencerschrock spencerschrock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.