Skip to content
This repository has been archived by the owner on Sep 1, 2023. It is now read-only.

Archive update #119

Closed
wants to merge 14 commits into from
Closed

Archive update #119

wants to merge 14 commits into from

Conversation

ineiti
Copy link
Member

@ineiti ineiti commented Aug 23, 2023

Build the images on github.

@ineiti ineiti self-assigned this Aug 23, 2023
@ineiti ineiti force-pushed the archive_update branch 7 times, most recently from 6dbd0c4 to 78b18de Compare August 24, 2023 06:42
@@ -10,7 +10,8 @@ on:
jobs:
build:
name: Lint, build and test
runs-on: ubuntu-18.04
runs-on: ubuntu-22.04
if: false
Copy link
Member

Choose a reason for hiding this comment

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

why not remove this if it is never run?

# with:
# context: backend
# push: true
# tags: c4dt/service-stainless-backend:latest
Copy link
Member

Choose a reason for hiding this comment

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

please remove the commented lines

volumes:
- ./backend/build/conode-1:/config
# volumes:
# - ./backend/build/conode-1:/config
Copy link
Member

Choose a reason for hiding this comment

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

please remove the commented lines

ports:
- "8080:8080"
# volumes:
# - ./webapp/src/assets:/webapp/assets
Copy link
Member

Choose a reason for hiding this comment

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

please remove the commented lines

@ineiti
Copy link
Member Author

ineiti commented Aug 24, 2023

Sorry, that was not supposed to be reviewed yet. I'm still in the middle of getting this one to work. And it's a mess!

@ineiti ineiti marked this pull request as draft August 24, 2023 15:32
@ineiti ineiti closed this Sep 1, 2023
@ineiti ineiti deleted the archive_update branch September 1, 2023 08:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants