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.

Sign up for GitHub

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

Jump to bottom

TIED-132 | Snyk improvements #365

Merged

charn merged 6 commits into main from TIED-132-snyk-improvements

Oct 27, 2023

Contributor

charn commented Oct 26, 2023

Update uWSGI configuration
Limit elasticsearch-dsl to 7.x
Bump all requirements
Update .pre-commit-config.yaml

Also add a redirect from the root to the api root (a common pattern in other projects).

charn requested a review from a team

October 26, 2023 13:36

codecov-commenter commented Oct 27, 2023 •

edited

Loading

Codecov Report

Merging #365 (833cd6a) into main (efe4305) will increase coverage by 0.00%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #365   +/-   ##
=======================================
  Coverage   95.55%   95.55%           
=======================================
  Files         104      104           
  Lines        5020     5021    +1     
=======================================
+ Hits         4797     4798    +1     
  Misses        223      223

Files	Coverage Δ
helerm/urls.py	`100.00% <100.00%> (ø)`

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

charn added 6 commits

October 27, 2023 13:29


          Ensure pip, setuptools, and wheel are up to date

43fdf3d

Refs TIED-132


          Limit elasticsearch-dsl < 8

b5ce4c7

Project uses elasticsearch 7

Refs TIED-132


          Bump all requirements

Refs TIED-132


          Update .pre-commit-config.yaml

48da944

- Sync tools with requirements-dev.txt
- Bump pre-commit-hooks

Refs TIED-132


          Update uWSGI configuration

70c38be

Include sensible configuration and add timeouts.

Refs TIED-132


          Redirect from root to api root

833cd6a

Refs TIED-132

charn force-pushed the TIED-132-snyk-improvements branch from cd21356 to 833cd6a Compare

October 27, 2023 10:29

sonarcloud bot commented Oct 27, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

nicobav approved these changes

View reviewed changes

Contributor

nicobav left a comment

LGTM

charn merged commit a189deb into main

4 checks passed

charn deleted the TIED-132-snyk-improvements branch

October 27, 2023 11:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet