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

fixed syntax issue in the scripts/deploy.sh #18

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dannytrue
Copy link

fixed syntax issue in the scripts/deploy.sh

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jjleigh
Copy link
Contributor

jjleigh commented Jul 24, 2023

Hi @dannytrue, I cannot merge this request because I no longer have write access. If you need this fix, you will have to fork the project.

@dannytrue
Copy link
Author

I already forked the project and fixed the issue. I want this fix to be available to all. Who has access to the repo?

@jjleigh
Copy link
Contributor

jjleigh commented Jul 25, 2023

Ok gotcha! Not sure at this point. It is my project, but I lost write access once I left AWS, so I suppose only the org owner has write access. Though I have no clue who that is right now.

@dannytrue
Copy link
Author

@jjleigh thank you for the quick response. I will figure out who is in charge of this project now.

@deepaksrinivasan27
Copy link

What was this issue please ?

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