Skip to content

Adding CI workflow and justfile - #57

Merged
simonteague6 merged 14 commits into
mainfrom
feat/ci-and-just
Aug 28, 2026
Merged

simonteague6 merged 14 commits into
mainfrom
feat/ci-and-just

Conversation

@simonteague6

Copy link
Copy Markdown
Contributor

Picking up this project again after a few months, it needs a CICD workflow, and the just file for easy codebase maneuverability via simple commands.

Copilot AI lite review requested due to automatic review settings August 28, 2026 04:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 21 out of 24 changed files in this pull request and generated 9 comments.

Comment on lines 60 to 64
def shortest_route(graph, start, end):
try:
return nx.shortest_path(graph, source=start, target=end, weight='weight')
except (nx.NetworkXNoPath, nx.NodeNotFound):
return nx.shortest_path(graph, source=start, target=end, weight="weight")
except nx.NetworkXNoPath, nx.NodeNotFound:
return None
Comment on lines 88 to 92
pairs = [
(floor_a, floor_b)
for floor_a, floor_b in zip(floors, floors[1:])
for floor_a, floor_b in itertools.pairwise(floors, floors[1:])
if floor_b == floor_a + 1
]
Comment thread app/templates/index.html
</select>
<br>
<br>
<button type="/directions">Get Directions</button>
Comment thread app/static/css/styles.css
margin: 10px 0;
padding-left: 20px;
}
.ateps-box li {
Comment thread app/static/css/styles.css
box-shadow: 0 2px 8px rgba(0, 0, 0, 0.1);
}

.direction-box h2 {
Comment thread app/static/css/styles.css
Comment on lines +149 to +160
#next--step-btn {
margin-top: 10px;
padding: 8px 12px;
background: #007bff;
color: white;
border: none;
border-radius: 5px;
cursor: pointer;
}
#next--step-btn:hover {
background: #0056b3;
}
Comment thread tools/validate_data.py
Comment thread docker-compose.prod.yml
Comment thread .github/workflows/ci.yml
Comment on lines +21 to +25
- name: "Install uv"
uses: astral-sh/setup-uv@v5
with:
enable-cache: true

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

using uv python install instead

simonteague6 and others added 2 commits August 28, 2026 09:36
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@simonteague6
simonteague6 merged commit 43a3505 into main Aug 28, 2026
1 check passed
@simonteague6
simonteague6 deleted the feat/ci-and-just branch August 28, 2026 14:52
@simonteague6

Copy link
Copy Markdown
Contributor Author

all other comments were general code concerns, not related to the changes I was making. the code concerns will be addressed in a later PR

@simonteague6

Copy link
Copy Markdown
Contributor Author

Linted and formatted code with just lint (just fix) and just format, passed just check

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