This repository has been archived by the owner on Jun 22, 2024. It is now read-only.
[FEATURE] Prepare for first release #49
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
roysti10
added this to the Make the website dynamic and ready to use for first release milestone
roysti10
force-pushed
the
feature-final
branch
from
b3a4f3e
to
759e567
Compare
* The index.html page has been made dynamic using the pycricbuzz package pls follow this is a breaking change , the prediction wont work just yet * [LICENSE] Added License doc strings as suggested by AGPLv3 License
roysti10
force-pushed
the
feature-final
branch
from
759e567
to
ca6ea78
Compare
roysti10
force-pushed
the
feature-final
branch
2 times, most recently
from
7d64a9d
to
f0dd2be
Compare
roysti10
force-pushed
the
feature-final
branch
from
6ecf0fb
to
52dfea5
Compare
Create DOCKERFILE.development
* dockerfiles Co-authored-by: Bastian <scientes12@gmail.com>
Co-authored-by: Daniel <54536136+DAN-329@users.noreply.github.com>
roysti10
changed the title
anihm136
suggested changes
Jan 5, 2021
| rm -rf /var/lib/apt/lists/* | ||
| COPY ./requirements.txt /app/requirements.txt | ||
| RUN pip install --no-cache-dir --user -r /app/requirements.txt; rm -r /tmp | ||
|
|
There was a problem hiding this comment.
Do you really need a separate compile image? It seems that you are just copying the installed dependencies over
There was a problem hiding this comment.
I think not,
I can probably use the one already referenced intially
| COPY ./app /app/app | ||
| COPY ./crawler /app/crawler | ||
| COPY ./scrapy.cfg /app/scrapy.cfg | ||
| ENV PYTHONPATH=/app |
There was a problem hiding this comment.
Not sure what's going on here with the pythonpath. Just changing WORKDIR to app should add the necessary modules to pythonpath
roysti10
force-pushed
the
feature-final
branch
from
ef58124
to
931ccfb
Compare
roysti10
force-pushed
the
feature-final
branch
from
bd4db11
to
544ac66
Compare
roysti10
added a commit
that referenced
this pull request
Jan 7, 2021
* [FEATURE] Made the page dynamic * [LICENSE] Added License doc strings as suggested by AGPLv3 License * predicts score * crawler complete * Website and crawler integrated * SEO tags * Cleaned query params(#57) * Dockerfiles set up(#59) * Improved UI Co-authored-by: Shreyas Raviprasad <shreyasraviprasad457@gmail.com> Co-authored-by: Sammith S Bharadwaj <54738235+SammithSB@users.noreply.github.com> Co-authored-by: scientes <34819304+scientes@users.noreply.github.com> Co-authored-by: Bastian <scientes12@gmail.com> Co-authored-by: Daniel <54536136+DAN-329@users.noreply.github.com> Co-authored-by: SammithSB <sammithsb@gmail.com>
roysti10
added a commit
that referenced
this pull request
Jan 7, 2021
* [FEATURE] Made the page dynamic * [LICENSE] Added License doc strings as suggested by AGPLv3 License * predicts score * crawler complete * Website and crawler integrated * SEO tags * Cleaned query params(#57) * Dockerfiles set up(#59) * Improved UI Co-authored-by: Shreyas Raviprasad <shreyasraviprasad457@gmail.com> Co-authored-by: Sammith S Bharadwaj <54738235+SammithSB@users.noreply.github.com> Co-authored-by: scientes <34819304+scientes@users.noreply.github.com> Co-authored-by: Bastian <scientes12@gmail.com> Co-authored-by: Daniel <54536136+DAN-329@users.noreply.github.com> Co-authored-by: SammithSB <sammithsb@gmail.com>
roysti10
added a commit
that referenced
this pull request
Jan 7, 2021
* [FEATURE] Made the page dynamic * [LICENSE] Added License doc strings as suggested by AGPLv3 License * predicts score * crawler complete * Website and crawler integrated * SEO tags * Cleaned query params(#57) * Dockerfiles set up(#59) * Improved UI * [TEST] Added basic tests Co-authored-by: Shreyas Raviprasad <shreyasraviprasad457@gmail.com> Co-authored-by: Sammith S Bharadwaj <54738235+SammithSB@users.noreply.github.com> Co-authored-by: scientes <34819304+scientes@users.noreply.github.com> Co-authored-by: Bastian <scientes12@gmail.com> Co-authored-by: Daniel <54536136+DAN-329@users.noreply.github.com> Co-authored-by: SammithSB <sammithsb@gmail.com>
roysti10
added a commit
that referenced
this pull request
Jan 7, 2021
* [FEATURE] Made the page dynamic * [LICENSE] Added License doc strings as suggested by AGPLv3 License * predicts score * crawler complete * Website and crawler integrated * SEO tags * Cleaned query params(#57) * Dockerfiles set up(#59) * Improved UI * [TEST] Added basic tests Co-authored-by: Shreyas Raviprasad <shreyasraviprasad457@gmail.com> Co-authored-by: Sammith S Bharadwaj <54738235+SammithSB@users.noreply.github.com> Co-authored-by: scientes <34819304+scientes@users.noreply.github.com> Co-authored-by: Bastian <scientes12@gmail.com> Co-authored-by: Daniel <54536136+DAN-329@users.noreply.github.com> Co-authored-by: SammithSB <sammithsb@gmail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The index.html page has been made dynamic using the pycricbuzz package
A New and updated crawler has been setup
Prediction works with a few minor bugs (will be fixed soon)
Cleaned frontend of the repo
Dockerfiles made
Tick the appropriate choice (Delete the rest):