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

Add support for joins, having, whereRaw and havingRaw #20

Closed
wants to merge 3 commits into from

Conversation

timohund
Copy link

No description provided.

@timohund timohund force-pushed the main branch 2 times, most recently from 537ab14 to cd036c5 Compare August 4, 2023 16:02
@norbybaru
Copy link
Owner

Hi @timohund, thanks for the PR. I am still trying to figure out why the test not running with your PR. Somehow build not passing the correct creds to access Timestream.

Could you try to create PR from a different branch than your main! maybe a feature branch to see if maybe it will run test with correct creds

@timohund
Copy link
Author

timohund commented Aug 7, 2023

@norbybaru Thanks for your feedback, i will open a new pr from a new branch.

@timohund timohund closed this Aug 7, 2023
@timohund
Copy link
Author

timohund commented Aug 7, 2023

@norbybaru regarding the credentials. I would guess that using the credentials in a pipeline of a pr is not possible at all because then i would be able to create a pr with code that could steal the credentials.

https://stackoverflow.com/questions/73866908/github-actions-main-repository-secret-not-picked-up-from-pull-request-build

Maybe it is possible to:

  • Run only unit tests on pr's
  • Run integration tests on a branch in your repository after a manual review

@timohund timohund mentioned this pull request Sep 1, 2023
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