Skip to content
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

Merging Backend and Backend-new #125

Merged
merged 43 commits into from
Nov 3, 2021
Merged

Merging Backend and Backend-new #125

merged 43 commits into from
Nov 3, 2021

Conversation

peeyush10234
Copy link

No description provided.

snapcat and others added 30 commits October 22, 2021 20:35
run: pylint --fail-under=1 $(find . -name "*.py" | xargs)
note: set the failure threshold as indicated above

actions/starter-workflows#636 (comment)
* Added CodeQL badge
* Added Pylint badge
* Minor edits: updated git clone steps
 All for the sake of a test badge.
@netlify
Copy link

netlify bot commented Nov 3, 2021

✔️ Deploy Preview for feature-hunt-25 ready!

🔨 Explore the source changes: bd5c8f2

🔍 Inspect the deploy log: https://app.netlify.com/sites/feature-hunt-25/deploys/6182cbcbe501550008ba34d4

😎 Browse the preview: https://deploy-preview-125--feature-hunt-25.netlify.app

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1417855388

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 72.054%

Totals Coverage Status
Change from base Build 1415139483: 0.0%
Covered Lines: 162
Relevant Lines: 188

💛 - Coveralls

@coveralls
Copy link

coveralls commented Nov 3, 2021

Pull Request Test Coverage Report for Build 1417992769

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 72.054%

Totals Coverage Status
Change from base Build 1415139483: 0.0%
Covered Lines: 162
Relevant Lines: 188

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1417855388

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 72.054%

Totals Coverage Status
Change from base Build 1415139483: 0.0%
Covered Lines: 162
Relevant Lines: 188

💛 - Coveralls

@jhnguye4 jhnguye4 merged commit 25488ac into main Nov 3, 2021
@snapcat snapcat requested review from snapcat and removed request for snapcat November 3, 2021 20:11
@snapcat snapcat linked an issue Nov 14, 2021 that may be closed by this pull request
Sumedh-Patkar pushed a commit to Sumedh-Patkar/feature-hunt that referenced this pull request Nov 28, 2023
* added pylint workflow w/artifacts

run: pylint --fail-under=1 $(find . -name "*.py" | xargs)
note: set the failure threshold as indicated above

actions/starter-workflows#636 (comment)

* Added CodeQL & Pylint badges, edits

* Added CodeQL badge
* Added Pylint badge
* Minor edits: updated git clone steps

* Lowered pylint failure theshold

* Update and rename build.yml to coveralls.yml

* Added test files to pylint.yml

* Updated py files to improve pylint score

* Fixed error in coveralls.yml

* Removed individual tests in pylint.yml

* Added pylint labels

* Added test workflow

* Update and rename nodejsTest.yml to nodejs.yml

* Update nodejs.yml

* Update nodejs.yml

* Added workflow build badge to README

* Added yet another workflow

 All for the sake of a test badge.

* Added test badge

* Revert "group25 workflows"

* Added post to Service.js and connected login form (incomplete)

* Added appropriate error messages and display for errors

* Added Sign up form

* Added error handling and redirecting for login and sign up

* fix to react compile warning

* Dashboard and tag edits are now limited by user

* New tags can now be saved in the database

* Adding Product Controller and Test Cases

* Adding more user auth and product auth test cases

* Changing gitignore

* Fixing test cases for user-auth class

* Removing test cases which are not used

* Commented out unused tests, added pylint comments

* Restored and commented out unused tests
* Added pylint skip comments

* Removing merging backend and backend-new into a single directory

* Increasing Pylint Score

* Increasing Pylint score

Co-authored-by: Leila Moran <lmmoran@ncsu.edu>
Co-authored-by: Emily Tracey <etracey@ncsu.edu>
Co-authored-by: peeyush10234 <ptaneja@ncsu.edu>
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.

Resolve Pylint Errors and cleanup backend code
4 participants