aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* pylintrc: skip many spurious WTForm no-member errorsBryan Newbold2020-05-261-0/+2
|
* HACK: try to squelch pylint in CIBryan Newbold2020-05-261-2/+2
| | | | | | | | | | | | | | | | | Gitlab CI is showing lint errors like: =================================== FAILURES =================================== 6316 _______________________ [pylint] tests/harvest_state.py ________________________ 6317 E: 19,11: hs.next is not callable (not-callable) 6318 E: 33,11: hs.next is not callable (not-callable) 6319 E: 19,11: hs.next is not callable (not-callable) [...] this is confusing as we use pipenv with a lock, so I should see the exact same errors locally. This commit is a hack to try and fix this and unbreak builds until we can debug further.
* sql: really don't double-dump requestsBryan Newbold2020-05-261-1/+0
| | | | | | I guess we were dumping 3 times originally; already had an earlier commit that removed one row from this README (that I copypaste to CLI every time)
* 2020-05-26 prod database size and statsBryan Newbold2020-05-262-0/+48
|
* HACK: skip pylint errors on lines that seem to be fineBryan Newbold2020-05-223-3/+3
| | | | | It seems to be an inadvertantly ugraded version of pylint saying that these lines are not-callable.
* run flake8 in CIBryan Newbold2020-05-221-0/+1
|
* pipenv: add flake8Bryan Newbold2020-05-222-183/+213
|
* Merge remote-tracking branch 'github/master'Bryan Newbold2020-05-223-11/+11
|\
| * Merge pull request #55 from cclauss/patch-1bnewbold2020-05-223-11/+11
| |\ | | | | | | Travis CI: Lint Python code for syntax errors and undefined names
| | * LICENSE.md: Properly capitalize brand namesChristian Clauss2020-05-141-4/+4
| | |
| | * Delete .travis.ymlChristian Clauss2020-05-141-6/+0
| | |
| | * Indentity is not the same this as equality in PythonChristian Clauss2020-05-141-2/+2
| | |
| | * Indentity is not the same this as equality in PythonChristian Clauss2020-05-141-5/+5
| | |
| | * python: 3.8Christian Clauss2020-05-131-0/+2
| | |
| | * Travis CI: Lint Python code for syntax errors and undefined namesChristian Clauss2020-05-131-0/+4
| |/
* | importers: clarify handling of ApiExceptionBryan Newbold2020-05-223-4/+10
| | | | | | | | | | | | | | | | One of these (in ingest importer pipeline) is an actual bug, the others are just changing the syntax to be more explicit/conservative. The ingest importer bug seems to have resulted in some bad file match imports; scale of impact is unknown.
* | ingest importer: don't use glutton matchesBryan Newbold2020-05-221-3/+3
| | | | | | | | | | | | | | Until reviewing I didn't realize we were even doing this currently. Hopefluly has not impacted too many imports, as almost all ingests use an external identifer, so only those with identifers not in fatcat for whatever reason.
* | retro CHANGELOG entry for python client library pypi packageBryan Newbold2020-05-141-0/+4
| |
* | version bump fatcat_openapi_clientBryan Newbold2020-05-147-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | This is to push a new package version to pypi with the newer codegen. The immediate motivation for this is to fix keyword errors with "async" parameters conflicting with "async" as a keyword in Python 3.7. I guess that means this is probably actually a breaking change to the python API, though not to the API itself? Hrm. All the other changes are typo corrections from the fatcat-openapi2.yml spec document.
* | Merge branch 'martin-fix-container-empty-search' into 'master'Martin Czygan2020-04-291-0/+4
|\ \ | | | | | | | | | | | | search: assume * when q is not set or empty See merge request webgroup/fatcat!51
| * | search: assume * when q is not set or emptyMartin Czygan2020-04-291-0/+4
| | | | | | | | | | | | An example would be a blank search from a container details page.
* | | Merge branch 'bnewbold-search-tweaks' into 'master'bnewbold2020-04-273-92/+132
|\ \ \ | |/ / |/| | | | | | | | tweaks to search result pages See merge request webgroup/fatcat!50
| * | web search: tweak release search result styleBryan Newbold2020-04-231-25/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is also back-ported from covid19.fatcat.wiki, though with some more tweaks on top. The changes are: - show original title if available (usually non-English) - move release_type label to title line suffix, and only show if not a "paper" - show publication status and withdrawl as text after the journal title, not as a label
| * | web search: improve indentation, fix missing div tagsBryan Newbold2020-04-232-67/+81
| |/ | | | | | | These are back-ported fixes from covid19.fatcat.wiki
* | Merge branch 'bnewbold-non-ident-fix' into 'master'Martin Czygan2020-04-243-6/+10
|\ \ | | | | | | | | | | | | | | | | | | fix ident=None broken links Closes #3 See merge request webgroup/fatcat!49
| * | web: fix ident=None broken linksBryan Newbold2020-04-233-6/+10
| |/ | | | | | | | | | | On web interface views for revisions, we had a bunch of broken links because the ident is "None". This commit fixes these by removing the links.
* | Merge branch 'martin-datacite-fix-parse-record-int' into 'master'bnewbold2020-04-244-2/+80
|\ \ | | | | | | | | | | | | datacite: fix type error See merge request webgroup/fatcat!48
| * | datacite: fix type errorMartin Czygan2020-04-224-2/+80
|/ / | | | | | | | | | | | | Up to now, we expected the description to be a string or list. Add handling for int as well. First appeared: Apr 22 19:58:39.
* | Merge branch 'martin-datacite-fix-release-contrib-raw-name-check-violation' ↵bnewbold2020-04-204-1/+86
|\ \ | | | | | | | | | | | | | | | | | | into 'master' datacite: fix a raw name constraint violation See merge request webgroup/fatcat!47
| * | datacite: fix a raw name constraint violationMartin Czygan2020-04-204-1/+86
| |/ | | | | | | | | | | | | It was possible that contribs got added which had no raw name. One example would be a name consisting of whitespace only. This fix adds a final check for this case.
* | Merge branch 'bnewbold-fix-changelog-es' into 'master'Martin Czygan2020-04-201-3/+12
|\ \ | |/ |/| | | | | fixes for changelog elasticsearch worker See merge request webgroup/fatcat!46
| * more changelog ES fixesBryan Newbold2020-04-171-4/+6
| |
| * ES changelog worker: fixes for ident; fetch update from API if neededBryan Newbold2020-04-171-2/+9
|/ | | | | The API fetch update may be needed for old changelog entries in the kafka feed.
* Merge branch 'bnewbold-py37-cleanups' into 'master'bnewbold2020-04-177-64/+39
|\ | | | | | | | | py37 cleanups See merge request webgroup/fatcat!44
| * example of starting to use format stringsBryan Newbold2020-04-171-12/+12
| |
| * pytest: ignore remaining deprecation warnings in 3rd party librariesBryan Newbold2020-04-171-0/+2
| |
| * consistently use raw string prefix for regexBryan Newbold2020-04-173-7/+7
| |
| * pipenv: update deps for python 3.7Bryan Newbold2020-04-172-45/+18
| | | | | | | | | | We had some pre-3.6 work arounds. Also seems like a reasonable time to update all depdencies to most recent versions.
* | Merge branch 'martin-changelog-to-es' into 'master'bnewbold2020-04-173-3/+41
|\ \ | | | | | | | | | | | | derive changelog worker from release worker See merge request webgroup/fatcat!43
| * | derive changelog worker from release workerMartin Czygan2020-04-173-3/+41
| | | | | | | | | | | | | | | Early versions of changelog entries may not have all the fields required for the current transform.
* | | Merge branch 'martin-changelog-release-types' into 'master'bnewbold2020-04-171-12/+17
|\ \ \ | | | | | | | | | | | | | | | | changelog: extend release_types considered documents See merge request webgroup/fatcat!42
| * | | changelog: limit typesMartin Czygan2020-04-161-5/+1
| | | | | | | | | | | | | | | | | | | | No partial docs (e.g. abstract), too generic components and entries, not HTML blogs.
| * | | changelog: extend release_types considered documentsMartin Czygan2020-04-161-10/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | according to release_rev.release_type, we have 29 values: fatcat_prod=# select release_type, count(release_type) from release_rev group by release_type; release_type | count -------------------+----------- abstract | 2264 article | 6371076 article-journal | 101083841 article-newspaper | 17062 book | 1676941 chapter | 13914854 component | 58990 dataset | 6860325 editorial | 133573 entry | 1628487 graphic | 1809471 interview | 19898 legal_case | 3581 legislation | 1626 letter | 275119 paper-conference | 6074669 peer_review | 30581 post | 245807 post-weblog | 135 report | 1010699 retraction | 1292 review-book | 96219 software | 316 song | 24027 speech | 4263 standard | 312364 stub | 1036813 thesis | 414397 | 0 (29 rows)
* | | | update prod statsBryan Newbold2020-04-177-0/+149
| |_|/ |/| |
* | | CHANGELOG entry for python 3.7Bryan Newbold2020-04-171-0/+4
| | |
* | | retro-active v0.3.2 changelog updatesBryan Newbold2020-04-172-2/+41
| | |
* | | Add missing packages to Dockerfile and CI fileBryan Newbold2020-04-162-3/+3
| | |
* | | ci: don't re-build/install commands if existingBryan Newbold2020-04-161-2/+2
| | |
* | | ci: only build postgres feature for dieselBryan Newbold2020-04-161-1/+1
| | |
* | | test-base DockerfileBryan Newbold2020-04-162-0/+51
| | | | | | | | | | | | Used to create bnewbold/fatcat-test-base image