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
Feature/GitHub actions #82
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a Simple change I wish we could do to release.bat.
Great job on automating, Gihub actions 💯 🚀
@@ -1,4 +1,4 @@ | |||
const ngrok = require('ngrok'); | |||
import * as ngrok from 'ngrok'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
@@ -1,14 +1,25 @@ | |||
import * as request from 'supertest'; | |||
import { expect } from 'chai'; | |||
import * as app from '../../index'; | |||
import {app} from '../../index'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool
infrastructure/demo-release.bat
Outdated
@@ -1,5 +1,5 @@ | |||
CALL cd .. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd move this set of comamnds to single bat.
And then jst have kind demo.bat which says relase.bat demo-covid19livebot
inside release bat it would be
CALL heroku container:push web --app=%1
CALL heroku container:release web --app=%1
CALL heroku open --app=%1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
letme try
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
5b28cd6
to
51acffc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
.github/workflows/reports.yml
Outdated
- name: Coveralls | ||
uses: coverallsapp/github-action@master | ||
with: | ||
github-token: ${{ secrets.GITHUB_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool 👍
@@ -1,5 +0,0 @@ | |||
CALL cd .. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually meant nor toremove this file,
But make it
release.bat demo-covid19livebot
and create a seperate one for prod
same way to be prod.release.bat
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@@ -1,3 +1,5 @@ | |||
[![Coverage Status](https://coveralls.io/repos/github/danbilokha/covid19liveupdates/badge.svg?branch=master)](https://coveralls.io/github/danbilokha/covid19liveupdates?branch=master) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't wait to see this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
34% 👍 Even though we're testing only 1 file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thant's because it hasn't been mocked, I suppose
51acffc
to
5d9aa01
Compare
5d9aa01
to
14fcbc2
Compare
No description provided.