Skip to content

UmdTask405_-DATA605_Spring2026_Ansible_Deployment-PR-1#419

Open
likhongomes wants to merge 58 commits into
gpsaggese:masterfrom
likhongomes:UmdTask405_-DATA605_Spring2026_Ansible_Web_Deployment
Open

UmdTask405_-DATA605_Spring2026_Ansible_Deployment-PR-1#419
likhongomes wants to merge 58 commits into
gpsaggese:masterfrom
likhongomes:UmdTask405_-DATA605_Spring2026_Ansible_Web_Deployment

Conversation

@likhongomes
Copy link
Copy Markdown

No description provided.

@protocorn
Copy link
Copy Markdown
Collaborator

Hi, please link your PR to your github issue

Copy link
Copy Markdown
Collaborator

@protocorn protocorn left a comment

Choose a reason for hiding this comment

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

This PR is incomplete.

It currently only includes a README file, but the expected project structure / template files are missing.

Please make sure your submission includes the full project template and that all required project files are added under your project directory

@likhongomes
Copy link
Copy Markdown
Author

what other file is missing? can you please specify? According to project description on Canvas the instrunction asks me to create the project repo, and add the readme, and I did just that. I thought this was just a Project Setup.

@protocorn
Copy link
Copy Markdown
Collaborator

Hi @likhongomes,

The assignment asks you to add the project template files to your project directory.

That said, the main goal of this assignment was just to help everyone get familiar with the workflow. Since this seems to have been a misunderstanding, I will still allot marks for it as long as you update the PR and fix the above.

@likhongomes likhongomes requested a review from protocorn April 9, 2026 05:38
@likhongomes
Copy link
Copy Markdown
Author

@protocorn I have added a Github Workflow file and an Ansible file to it, the workflow lints an Ansible playbook. That should satisfy the requirement for now.

Thanks

@likhongomes
Copy link
Copy Markdown
Author

@protocorn & @gpsaggese please take a look at this PR, I believe that's everything for the Final Project, let me know if I am missing something, once everything looks good, I will make a video demo of it.

Thanks

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