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

chore(wren-ai-service): manual running pytest to avoid auth issue from forked repo #734

Merged
merged 2 commits into from
Oct 9, 2024

Conversation

paopa
Copy link
Member

@paopa paopa commented Oct 8, 2024

We encountered an issue similar to the one documented in this GitHub Actions run. To address this, we have added the workflow_dispatch event to the GitHub Actions workflow. This change allows maintainers to manually trigger the pytest job from the Actions page, enabling contributors to validate their changes more effectively.

This enhancement provides a temporary solution to the issue at hand. In the future, we may explore additional methods to improve our testing workflow further.

@paopa paopa changed the title chore(wren-ai-service): manual running pytest to validate auth issue from forked repo chore(wren-ai-service): manual running pytest to avoid auth issue from forked repo Oct 8, 2024
@paopa paopa marked this pull request as ready for review October 8, 2024 07:52
@paopa paopa requested a review from cyyeh October 8, 2024 07:52
@paopa paopa added the module/ai-service ai-service related label Oct 8, 2024
Copy link
Member

@cyyeh cyyeh left a comment

Choose a reason for hiding this comment

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

LGTM

@cyyeh cyyeh merged commit f87d5a5 into main Oct 9, 2024
8 checks passed
@cyyeh cyyeh deleted the chore/manual-run-pytest branch October 9, 2024 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module/ai-service ai-service related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants