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

Initial implementaion of the Datetime server #1

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

Conversation

MohamedFadel01
Copy link
Member

This pull request introduces the full project setup, including:

  1. Datetime Server Implementations:

    • Two HTTP server implementations that return the current date and time.
      • One using the Go net/http package.
      • Another using the Gin framework.
    • Both servers respond with either plain text or JSON format, depending on the 'Accept' header.
  2. Dockerization:

    • Dockerfiles for both net/http and Gin implementations, enabling easy containerization and deployment.
    • A docker-compose.yml file to orchestrate both containers.
  3. Makefile:

    • Automates key development tasks such as:
      • Running go fmt for formatting.
      • Linting using golangci-lint.
      • Building the Go binaries.
      • Docker image creation and container management.
  4. GitHub Actions CI Pipeline:

    • A workflow to automate:
      • Linting with golangci-lint to enforce code quality.
      • Formatting the code using go fmt.
      • Running unit tests and uploading test coverage results.
    • The workflow triggers on pull requests and pushes to the main branch.

This setup ensures the project is ready for development, testing, and deployment with automated code quality checks and containerization in place.

Comment on lines +3 to +9
on:
pull_request:
branches:
- main
push:
branches:
- main

Choose a reason for hiding this comment

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

let the pr run in push on all branches

Choose a reason for hiding this comment

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

ignore this file

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.

DateTime Server - Mohamed Fadel
2 participants