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.

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

[WIP] python-upgrade cr responses part 1 #10656

Closed
wants to merge 342 commits into from
Closed

[WIP] python-upgrade cr responses part 1 #10656

wants to merge 342 commits into from

Conversation

felliott
Copy link
Member

Purpose

Responding to CR, checking ci results

Changes

Mostly f-strings so far

QA Notes

Nope. Placeholder PR

Documentation

I shall not fear. Fear is the blah, blah, blah

Side Effects

Nausea, vomiting, the other thing.

Ticket

https://openscience.atlassian.net/browse/ENG-5681

felliott and others added 30 commits March 18, 2024 16:05
@felliott
Merge branch 'feature/dep-group-formatting' into feature/python-upgrade
0196725 
 [ENG-5366]
 Closes CenterForOpenScience#10559
@ly-mariia @felliott
replace bulk_update module with a built-in .bulk_update
867616d
@ly-mariia @felliott
update versions of dj-celery-beat and drf-csv modules
c1e918d
@felliott
update django* requirements
df7e5aa
@felliott
Merge branch 'feature/dep-group-django' into feature/python-upgrade
aad7c7a 
 [ENG-5330]
 Closes CenterForOpenScience#10558
@opaduchak @felliott
bumped dev dependencies
1812243
@opaduchak @felliott
fixed flake8 warnings
473cd97
@felliott
Merge branch 'feature/dep-group-dev' into feature/python-upgrade
3dc4b04 
 [ENG-5336]
 Closes CenterForOpenScience#10566
@ly-mariia @felliott
bump version of gevent
1c47f33
@ly-mariia @felliott
update dep versions for transitions, pillow, geoip2
72b1aae 
 - bump version for the transitions module;
 - add new modules: pillow, geoip2;
@ly-mariia @felliott
specify str encoding in nameparser
a4afa04 
 - added a new parameter to specify the encoding of strings in
   constants for nameparser module;
@ly-mariia @felliott
remove unused Pystache module
5b4539c
@ly-mariia @felliott
replace maxminddb-geolite2 to more modern geoip2;
faa1f23
@ly-mariia @felliott
remove python-magic
e7f1084
@felliott
downgrade geoip2 to avoid setuptools build failure
64a8561
@felliott
Merge branch 'feature/dep-group-catchall' into feature/python-upgrade
78379ea 
 ENG-5340
 Closes CenterForOpenScience#10567
@opaduchak @felliott
removed _app_ctx_stack and _request_ctx_stack
e46aa12
@opaduchak @felliott
fied uritemplate.py -> uritemplate
5b28d61
@opaduchak @felliott
fixed future version of ipdb
39d7dc2
@felliott
Merge branch 'feature/fix-flask-context' into feature/python-upgrade
57f62fc 
 [ENG-5456]
 Closes CenterForOpenScience#10573
@felliott
remove egap assets missed from prior purge
61e16db
@felliott
repoint dv-client-pin pin to COS repo
2f97bea
@felliott
repoint mendeley-python-sdk pin to COS repo
33aecfd
@felliott
repoint djelme pin to COS repo
d8ed3a9
@felliott
replace mdx_del_ins fork with markdown-del-ins from pypi
054ba6b
@felliott
Merge branch 'feature/misc-updates' into feature/python-upgrade
5391352 
 * update github deps to pull from COS repos
@opaduchak @felliott
updated furl usage
0e9eb50
@opaduchak @felliott
added comments in locations that may be affected by furl upgrade
c7864aa
@opaduchak @felliott
added reasoning behind urrlib3 version
8a4ac00
@felliott
replace furl / op with .add(path=); minor typo fix
e656848
felliott and others added 28 commits June 8, 2024 01:33
@felliott
Merge branch 'feature/fix-token-redirects' into feature/python-upgrade
744eab2 
 [ENG-5746]
 Closes CenterForOpenScience#10637
@opaduchak
fixed s3 failures when setting up addon (not when browsing files)
8c20bb7
@felliott
Merge branch 'feature/s3-setup-fix' into feature/python-upgrade
f7aeefe 
 [ENG-5780]
 Closes CenterForOpenScience#10639
@opaduchak @felliott
fixed box addon setup
2f0226c
@opaduchak @felliott
removed state as it wasn't needed
9a3554f
@felliott
Merge branch 'feature/box-client-fix' into feature/python-upgrade
64cf539 
 [ENG-5796]
 Closes CenterForOpenScience#10640
@felliott
Merge remote-tracking branch 'cos/develop' into feature/python-upgrade
353c536 
 * Include pyupgrade changes for py3.12
@felliott
i forgot to flake
26e7b9d
@felliott
Merge branch 'feature/shame-on-me' into feature/python-upgrade
6a08399
@opaduchak @felliott
switch to poetry for dep management
8331d96 
 * made poetry resolve dependencies

 * fixed broken mendeley and dataverse-client-python dependencies to
   resolve alongside osf deps

 * added poetry to Dockerfile and CI

 * converted Dockerfile to multi-stage build in order to reduce image
   size
@felliott
Merge branch 'feature/poetry' into feature/python-upgrade
af0b559 
 [ENG-5686]
 Closes CenterForOpenScience#10631
@felliott
remove unused files
11190f0
@felliott
Merge branch 'feature/remove-obsolete-file' into feature/python-upgrade
9a88b30
@felliott
Merge remote-tracking branch 'cos/develop' into feature/python-upgrade
1458262 
 [ENG-5832]
@felliott
Merge remote-tracking branch 'cos/develop' into feature/python-upgrade
f43096f
@opaduchak
fixed project metrics not being collected
b013d7b
@opaduchak
modernized if statement
7b4126b
@felliott
Merge branch 'feature/fix-metrics' into feature/python-upgrade
f47ed16 
 [ENG-5772]
 Closes CenterForOpenScience#10651
@felliott
first pass on CR - mostly f-strings
e97f8e4
@felliott
second pass on CR - more f-strings and tests
393dfce
@felliott
remove unnecessary kwarg
9b4b6d6
@felliott
fourth pass on CR - more f-strings and tests
3fe7975
@felliott
Merge remote-tracking branch 'cos/develop' into feature/cr-responses-…
297df85 
…part-1

 Merge changes from develop + pyupgrades
@felliott
missed cr response
670e2f5
@felliott
fun with f-strings
33af739
@felliott
cr round 5: mostly f-strings and comments
7d7e46b
@felliott
forgot to f them up
d0e77f9
@felliott
fix furl usage + use ensure_str
176eafa
@felliott felliott closed this Jul 16, 2024
@felliott felliott deleted the feature/cr-responses-part-1 branch July 24, 2024 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@felliott @ly-mariia @opaduchak @cslzchen