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

move extension to ./pgmq-extension #266

Merged
merged 12 commits into from
Jun 18, 2024
Merged

move extension to ./pgmq-extension #266

merged 12 commits into from
Jun 18, 2024

Conversation

ChuckHend
Copy link
Member

Moves all the extension files to ./extension. This will hopefully help keep the overall project (pg extension, client libraries) more organized. This should hopefully make it easier to separate the various CI jobs for each sub-project in this repo.

@ChuckHend
Copy link
Member Author

I'm thinking maybe wait to merge this until after @theory PRs merge so that there are not massive merge conflicts.

@ChuckHend ChuckHend marked this pull request as ready for review June 15, 2024 19:15
@v0idpwn
Copy link
Collaborator

v0idpwn commented Jun 16, 2024

Maybe overthinking but could call it pgmq-extension, envisioning possible extra extensions in the future, like a different one for topics/routing/fan-out, or bgw/rust, etc

@ChuckHend
Copy link
Member Author

pgmq-extension

Good idea. I'll push that up.

@ChuckHend ChuckHend changed the title move extension to ./extension move extension to ./pgmq-extension Jun 16, 2024
@ChuckHend ChuckHend requested a review from v0idpwn June 17, 2024 23:21
@ChuckHend ChuckHend merged commit 676fc0a into main Jun 18, 2024
13 checks passed
@ChuckHend ChuckHend deleted the extension-dir branch June 18, 2024 12:07
Copy link
Contributor

@theory theory left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

3 participants