-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Error when using Action: Pipenv is not intended to work under the root directory, please choose another path #11
Comments
Same problem for me too. When I try to build |
Seems to be related to the release of a new version of pipenv a few hours ago. Changing the diff --git a/Dockerfile b/Dockerfile
index 8387bbb..0ba2940 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -6,7 +6,7 @@ ENV PYTHONUSERBASE $PYROOT
FROM base AS builder
-RUN pip install pipenv && \
+RUN pip install 'pipenv==2018.11.26' && \
apt-get update -y && \
apt-get install -y git I don't know the Python ecosystem enough to tell if the new version just surfaced an issue that was previously just ignored though. Better if someone could look into changing the problematic line, so this Action can keep using the latest version of |
Adding
|
see benjefferies#11 for more details
@bengry maybe you can create a pull request for your workaround? |
I don't really like my workaround for not using the latest package. Let's see if @benjefferies has anything to say here. In the meantime I forked the repo (and added the workaround) for my use of it, but I don't think it should be merged back to this repo as-is. |
Good work with the investigation. It's a good chance it could have something to do with the latest version of pipenv being pushed. I can pin the pipenv version to unblock you guys for now and look at it this evening |
That's merged and fixed now guys. I will take a look at what's changed in the latest pipenv that's causing these warnings |
I can confirm the workaround is doing it's job! :) |
Thanks. Please create a tag for the latest |
thanks @benjefferies ! |
Ah, you reminded me of something I meant to do before getting caught up looking at the issue 👍. Done https://github.com/benjefferies/branch-protection-bot/releases/tag/1.0.3 |
I've raised a PR with these changes and followed up this issue with pipenv. Once it's resolved I'll merge |
In the past 24 hours or so there's been an error when trying to run the Action:
This of course fails the whole workflow.
Here are the raw logs for both steps that
benjefferies/branch-protection-bot@1.0.2
is part of:2_Build benjefferiesbranch-protection-bot@1.0.2.txt
11_Enable include administrators branch protection.txt
The error seems to be this:
The text was updated successfully, but these errors were encountered: