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 pgrx test and reorganize PGXS tests #17

Merged
merged 1 commit into from
Apr 23, 2024
Merged

Add pgrx test and reorganize PGXS tests #17

merged 1 commit into from
Apr 23, 2024

Conversation

theory
Copy link
Member

@theory theory commented Apr 22, 2024

Move the PGXS tests to a subdirectory separate testing PGXS from testing zip vs. Git bundling in cicd.yml. Then add a pgrx "hello world" project in test/pgrx and test it, too, including pg_regress tests.

Update /etc/sudoers and the PATH environment variable to allow otherwise unprivileged users to do what they need to do. Also remove nobody.

Copy-edit the README a bit and explicitly set the PATH environment variable in the Dockerfile, because /etc/profile.d scripts apparently don't run in GitHub workflows.

@theory theory added the enhancement New feature or request label Apr 22, 2024
@theory theory self-assigned this Apr 22, 2024
@theory theory marked this pull request as ready for review April 22, 2024 22:30
@theory theory force-pushed the pgrx-test branch 10 times, most recently from 4df357b to 0c146aa Compare April 23, 2024 19:07
Base automatically changed from pgrx to main April 23, 2024 19:18
Move the PGXS tests to a subdirectory separate testing PGXS from testing
zip vs. Git bundling in `cicd.yml`. Then add a pgrx "hello world"
project in `test/pgrx` and test it, too, including `pg_regress` tests.

Update `/etc/sudoers` and the `PATH` environment variable to allow
otherwise unprivileged users to do what they need to do. Also remove
`nobody`.

Copy-edit the README a bit and explicitly set the PATH environment
variable in the `Dockerfile`, because `/etc/profile.d` scripts
apparently don't run in GitHub workflows.
@theory theory merged commit ca4c927 into main Apr 23, 2024
54 checks passed
@theory theory deleted the pgrx-test branch April 23, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant