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

Pre-release merge for minor release v0.5.3 #235

Merged
merged 106 commits into from
Mar 12, 2019
Merged

Pre-release merge for minor release v0.5.3 #235

merged 106 commits into from
Mar 12, 2019

Conversation

mhl10
Copy link
Contributor

@mhl10 mhl10 commented Mar 7, 2019

This upgrades the arxiv-base dependency to 0.15.2, which in search updates the page footer to the four-column version.

@mhl10 mhl10 requested a review from eawoods March 7, 2019 14:32
@mhl10 mhl10 requested a review from erickpeirson March 7, 2019 14:34
@JaimieMurdock
Copy link
Contributor

The mypy failure is likely related to the fixes I just released into task/ARXIVNG-1971. I'll check out the code and test in the next 30 minute to update.

I think the develop branch is protected for direct commits, but if it is related, then a git cherry-pick 9a1389da21352c77c74b17617287406554783eb0 should pull out the fixes we'd need.

@mhl10
Copy link
Contributor Author

mhl10 commented Mar 7, 2019

The mypy failure is likely related to the fixes I just released into task/ARXIVNG-1971. I'll check out the code and test in the next 30 minute to update.

I think the develop branch is protected for direct commits, but if it is related, then a git cherry-pick 9a1389da21352c77c74b17617287406554783eb0 should pull out the fixes we'd need.

Thanks Jaimie!

@JaimieMurdock
Copy link
Contributor

It seems like the issue was resolved by arXiv/arxiv-base#106

pipenv run mypy -p search | grep -v "test*" | grep -v "defined here"

returns nothing, which indicates a success.

mypy ignores for a variety of errors
@JaimieMurdock
Copy link
Contributor

JaimieMurdock commented Mar 7, 2019

I think the latest mypy fixes should make it all work, but it also looks like there's more than just a version bump on arxiv-base that's going into master, based on the number of tickets and commits in the history here. Perhaps we should audit first and get some more release notes together?

Oddly the diff only seems to have the most recent changes, but it's acting like 100+ commits will be merged.

Copy link
Contributor

@erickpeirson erickpeirson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forward!

Adding size and start to OpenAPI parameters list ARXIVNG-1895 ARXIVNG-1896
@mhl10 mhl10 merged commit 845c0e8 into master Mar 12, 2019
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.

4 participants