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.

Sign up for GitHub

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

Jump to bottom

Add Dockerfile php8.1 #29

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add Dockerfile php8.1 #29

wants to merge 3 commits into from

Conversation

BehroozBvk
Copy link

Changes Made Describe

the changes made to the codebase.

Checklist

  • [✅] I have tested the changes locally.
  • [✅] I have added the necessary documentation (if appropriate).
  • [✅] I have updated the README (if appropriate).
  • [✅] I have merged the latest changes from the main branch.

@BehroozBvk BehroozBvk changed the title Initial Dockerfile php8.1 Add Dockerfile php8.1 Mar 29, 2023
jigsaw373
jigsaw373 suggested changes Mar 29, 2023
View reviewed changes
Copy link
Contributor

@jigsaw373 jigsaw373 left a comment

Choose a reason for hiding this comment

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

Please remove some usefulness lines as well as files

03-PHP/8.1/Dockerfile Outdated Show resolved Hide resolved
03-PHP/8.1/Dockerfile Show resolved Hide resolved
03-PHP/8.1/Dockerfile Show resolved Hide resolved
03-PHP/8.1/config/nginx.conf Outdated Show resolved Hide resolved
BehroozBvk
BehroozBvk commented Mar 30, 2023
View reviewed changes
Copy link
Author

@BehroozBvk BehroozBvk left a comment

Choose a reason for hiding this comment

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

change Dockerfile and nginx.conf

BehroozBvk

This comment was marked as duplicate.

Sign in to view

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jigsaw373 jigsaw373 jigsaw373 requested changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BehroozBvk @jigsaw373