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

Publish lambda layer in all AWS regions #31

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Conversation

qpre
Copy link
Contributor

@qpre qpre commented Jul 24, 2024

see #30

@dasfmi
Copy link
Collaborator

dasfmi commented Aug 5, 2024

hey @qpre , I haven't seen your PR till now, sorry for the late show. You have added the regions in the dev/test part of the CI, the release CI file is the right place to do so, happy to merge and release to other regions once you update the file.

@qpre
Copy link
Contributor Author

qpre commented Oct 4, 2024

Hello @SollyzDev, I am not sure I follow you, there are only one CI file that I can find in this repository and the list of regions was updated for the publish_to_production part of it.

please show me where I should do other modifications if needed :)

@dasfmi
Copy link
Collaborator

dasfmi commented Oct 7, 2024

Looks like they are in the right place, just no need to update the ones for dev, the changes for publish_to_production are the only ones needed, we can revert the dev ones (74:76) to be as before.

@dasfmi dasfmi merged commit 5cff4a4 into axiomhq:main Nov 4, 2024
7 checks passed
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