Skip to content

Update README#238

Open
AsgerT10 wants to merge 5 commits into
developfrom
readmeUpdate
Open

Update README#238
AsgerT10 wants to merge 5 commits into
developfrom
readmeUpdate

Conversation

@AsgerT10
Copy link
Copy Markdown
Collaborator

Update README

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented May 14, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Comment thread README.md Outdated
go run ./cmd/api # API on http://localhost:5001/api
```

To reset the database, run `rm /tmp/minitwit.db`.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is not true anymore

Comment thread README.md Outdated

## Run with Docker Swarm

Requires Docker with Swarm mode and the Compose plugin.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Is it correct to say "plugin"?

Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md
make all # lint + static analysis
```

## Deploy to Production
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Not correct

Comment thread README.md Outdated

We use a feature-branch workflow with mandatory PR review.

1. Branch off `develop`: `git checkout -b feat/<short-description>`.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

feat/ is not a pattern we have used

@sonarqubecloud
Copy link
Copy Markdown

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