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

Update reentry to v1.2.0 for release branch #1440

Conversation

sphuber
Copy link
Contributor

@sphuber sphuber commented Apr 18, 2018

Fixes #1429

Update reentry to v1.2.0 which fixes the issue with entry points not being registered.
This currently contains the alpha release v1.2.0a9, but once it is confirmed to work
in all relevant environments and branches the actual version v1.2.0 will be released
and I will update this PR with the actual release

@codecov-io
Copy link

codecov-io commented Apr 18, 2018

Codecov Report

Merging #1440 into release_v0.12.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           release_v0.12.0    #1440   +/-   ##
================================================
  Coverage            54.42%   54.42%           
================================================
  Files                  246      246           
  Lines                32440    32440           
================================================
  Hits                 17655    17655           
  Misses               14785    14785

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0dda0c0...5f46591. Read the comment docs.

@ltalirz ltalirz self-requested a review April 18, 2018 21:31
ltalirz
ltalirz previously approved these changes Apr 18, 2018
Copy link
Member

@ltalirz ltalirz left a comment

Choose a reason for hiding this comment

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

Finally!
So, will this PR be updated (in which case one will need to re-approve)?

Also, can one get rid of the REENTRY_PINNED variable?

@sphuber
Copy link
Contributor Author

sphuber commented Apr 18, 2018

I can temporarily lift the setting that rebasing or adding more commits renders the approval stale, so then reapproval is not necessary. Regarding the REENTRY_PINNED variable, that is something I think @giovannipizzi request so you would have to take that up with him.

Also, we still need to make sure that this alpha version fixes the problem in all relevant environments. I made a list in the issue #1429 . I know you use conda, could you please verify that this branch works with conda?

@ltalirz
Copy link
Member

ltalirz commented Apr 18, 2018

@sphuber Rico asked me to check out reentry with conda a few hours ago and it seemed to work (i.e. data plugin registered by aiida-siesta became available after pip install aiida-siesta).
This is on release_v0.12.0. I have no reason to believe that anything would change on workflows.

@giovannipizzi
Copy link
Member

I asked for the REENTRY_PINNED variable (or any equivalent mechanism) because the version of reentry has to be specified in two different locations. It has nothing to do with this issue, so I would advise to keep it. @sphuber did you have a chance to check if this fixes also the Workflows branch?

@sphuber
Copy link
Contributor Author

sphuber commented Apr 19, 2018

@giovannipizzi yes, you can check the issue #1429 where I added a list of checkboxes of environments that I have tested. We are just missing the check for both branches on Mac OS X. Could you maybe give a quick look to that on your machine?

Starting `v10.0.0`, pip supports PEP 518 through the `pyproject.toml`
file, which can be used to define dependencies that are required by the
build process. We chose to use this mechanism over the `setup_requires`
keyword in the `setup` call of `setup.py`, because that causes problems
on older standard python version on Mac OS X as that goes through setuptools
instead of pip.

To ensure that the version of `reentry` specified in the `pyproject.toml`
and the `setup_requirements.py`, I also add a pre-commit test.
@sphuber sphuber force-pushed the fix_1429_reentry_version_release_branch branch 3 times, most recently from 12e796e to c72ebf5 Compare April 19, 2018 14:10
The new build procedure employing the pyproject.toml file relies
on pip being at least v10.0.0, so we test that this requirement
is satisfied in the setup.py or exit with a warning
@sphuber sphuber force-pushed the fix_1429_reentry_version_release_branch branch from c72ebf5 to 5f46591 Compare April 19, 2018 14:16
@sphuber sphuber merged commit 989e091 into aiidateam:release_v0.12.0 Apr 19, 2018
@sphuber sphuber deleted the fix_1429_reentry_version_release_branch branch April 19, 2018 14:41
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.

5 participants