Skip to content

feat/pre deployment setup#2

Merged
tnagorra merged 10 commits into
mainfrom
feat/pre-deployment-setup
Sep 25, 2025
Merged

feat/pre deployment setup#2
tnagorra merged 10 commits into
mainfrom
feat/pre-deployment-setup

Conversation

@thenav56
Copy link
Copy Markdown
Member

@thenav56 thenav56 commented Sep 8, 2025

@thenav56 thenav56 force-pushed the feat/pre-deployment-setup branch from 43a6c4d to e833b35 Compare September 8, 2025 18:01
@thenav56 thenav56 force-pushed the feat/pre-deployment-setup branch from f8c261a to dc55749 Compare September 10, 2025 02:03
@tnagorra tnagorra marked this pull request as ready for review September 10, 2025 14:51
Comment thread scripts/get_commit_hash.sh Outdated
Comment on lines +1 to +3
#!/bin/bash

#!/bin/bash
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This shebang is duplicated.

Comment thread README.md Outdated

Clone and pull all submodules
```bash
git clone git@github.com:mapswipe/mapswipe-deploy.git --branch feat/pre-deployment-setup
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We might need to change this.

Comment thread README.md
Comment on lines +38 to +39
```bash
task --list-all
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We might need to add that we are using https://taskfile.dev/

@tnagorra tnagorra force-pushed the feat/pre-deployment-setup branch from a554047 to 527d07e Compare September 25, 2025 06:39
@tnagorra tnagorra merged commit 808f51e into main Sep 25, 2025
1 check passed
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