Skip to content

Added Makefile#40

Merged
Seluj78 merged 32 commits into
masterfrom
makefile
Jan 9, 2020
Merged

Added Makefile#40
Seluj78 merged 32 commits into
masterfrom
makefile

Conversation

@Seluj78

@Seluj78 Seluj78 commented Jan 7, 2020

Copy link
Copy Markdown
Owner

Do not merge until #35 is merged

@gmorer can you help me on this ? I need color, fancy compiles and stuff

@Seluj78 Seluj78 requested a review from gmorer January 7, 2020 12:10
@Seluj78 Seluj78 self-assigned this Jan 7, 2020
@Seluj78 Seluj78 marked this pull request as ready for review January 7, 2020 13:29
Comment thread Makefile Outdated
Comment thread Makefile Outdated
@Seluj78 Seluj78 merged commit 95fffda into master Jan 9, 2020
@Seluj78 Seluj78 deleted the makefile branch January 9, 2020 20:57
gmorer added a commit that referenced this pull request Feb 21, 2020
* Adding the front and moving the back in his directory

* Renamed front -> frontend

* Removed node modules and added folder to gitignore

* Moved files to frontend folder

* Moved black config to root

* Started on a Makefile

Will fix #38

* Updated makefile and moved files back to normal dir

* Removed useless file

* Some adjustments

* Added venv source

* New travis setup

* fix Makefile on the source

* npm install fix

* Should be working

* Testing

* Added back build

* Renamed folders

* Moved home react route to base init file

* Added more specifications for 3.7 python and source

* Testing new stuff for venv

* Testing with caps

* Added cors

* [FIX] error 404 on /

* Trying new travis install technique (not using the makefile)

* Forgot the folder

* Changed makefile prereqs

* Removed useless install on tests

* [FIX] Npm wasn't fully installed after last change

* Added test to see if build was correctly creared

* [FIX] Added a env var test for CI so python doesnt install on travis

* [FIX] build backend

* Apply suggestions from code review

Co-authored-by: gmorer <gmorer@users.noreply.github.com>
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