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

Removed Flask-Common. Fixed #442. #447

Merged
merged 1 commit into from
May 6, 2018

Conversation

javabrett
Copy link
Contributor

@javabrett javabrett commented May 5, 2018

Fixed #442.

Pipfile.lock Outdated
"sha256": "e2d593192acadabf3463785be0dc3a1e163d41713ca8d4cd53d4e99c03d927a4"
"sha256": "55b1275da9acdf1b325c78f2be89df79f5435330ef7333b8caf9535350c24a72"
},
"host-environment-markers": {
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we want these. Can you regenerate the Pipfile.lock without them?

@javabrett
Copy link
Contributor Author

Pipfile.lock was updated via pipenv lock. According to pypa/pipenv#753 (re)generation of host-environment-markers is intended and ongoing.

Is there some switch for pipenv to avoid regeneration, or should pipenv be avoided, or the entries removed by hand, or pypa/pipenv#753 re-raised?

@nateprewitt
Copy link
Contributor

@sigmavirus24, unless it’s been added in the last month or so, I don’t believe there’s any way to disable those.

@javabrett, you’ll want to leave it for now I think. The Pipfile.lock shouldn’t be modified by hand. We’ve typically had a maintainer generate the lockfile to keep some level of consistency. I don’t see a huge reason that’s necessary though, outside of someone else’s system info being stored.

@javabrett
Copy link
Contributor Author

I could just revert the Pipfile.lock and let a maintainer lock, or feel free to edit my commit for that, then a maintainer can lock.

I am fairly new to pipenv and have been noticing a fair bit of diff churn noise when locking e.g. order of elements.

@javabrett javabrett force-pushed the 442-remove-flask-common branch from 662def3 to e987ecf Compare May 6, 2018 04:29
@javabrett
Copy link
Contributor Author

I since discovered pypa/pipenv@ecf7842 , and that my pipenv was as-shipped with my CPython and hopelessly out-of-date, so upgraded:

$ which pipenv
/Library/Frameworks/Python.framework/Versions/3.6/bin/pipenv

$ pipenv --version
pipenv, version 8.3.2

$ pip install -U pipenv

$ pipenv --version
pipenv, version 11.10.1

... and pipenv lock no-longer adds that host env.

@sigmavirus24 @nateprewitt please re-review.

@sigmavirus24 sigmavirus24 merged commit 7a50a02 into postmanlabs:master May 6, 2018
@sigmavirus24
Copy link
Contributor

Of note, Flask-common has been unarchived so it could be updated for Flask 1.0, but this is causing direct pain for lots of folks, so I intend to merge this and release it. We can add flask-common back later.

@javabrett javabrett deleted the 442-remove-flask-common branch May 6, 2018 20:54
virgiliojr94 pushed a commit to virgiliojr94/httpbin-chat2desk that referenced this pull request Jul 1, 2023
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.

3 participants