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.

Sign up for GitHub

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

Jump to bottom

pipenvアップデート処理を一時的に削除 #4030

Merged
merged 1 commit into from
Jun 10, 2024
Merged

pipenvアップデート処理を一時的に削除 #4030

merged 1 commit into from
Jun 10, 2024

Conversation

massongit
Copy link
Member

@massongit massongit commented Jun 10, 2024 •
edited
Loading

https://github.com/dev-hato/hato-bot/actions/runs/9437719483/job/25994118763

#36 73.30 [pipenv.exceptions.InstallError]: Collecting sudden-death@ git+https://github.com/dev-hato/sudden-death@ (from -r /tmp/pipenv-dne7vscx-requirements/pipenv-cvolrgp9-reqs.txt (line 21))
#36 73.30 [pipenv.exceptions.InstallError]: ERROR: The URL 'git+https://github.com/dev-hato/sudden-death@' has an empty revision (after @) which is not supported. Include a revision after @ or remove @ from the URL.
#36 73.30 ERROR: Couldn't install package: {}
#36 73.30  Package installation failed...

pipenvを2024.0.0にアップデートするとsudden-deathのインストールに失敗するようなので、この問題を解決できるまでpipenvのアップデート処理を削除します。

@massongit massongit mentioned this pull request Jun 10, 2024
Merged
@massongit massongit merged commit f1699aa into develop Jun 10, 2024
40 checks passed
@massongit massongit deleted the rm_update_pipenv branch June 10, 2024 11:25
massongit
massongit commented Jun 10, 2024
View reviewed changes
scripts/pr_format/pr_format/install_pipenv.sh
pip install "${package_name}==${new_version}"
exit 1
fi
fi
Copy link
Member Author

Choose a reason for hiding this comment

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

test

massongit
massongit commented Jun 10, 2024
View reviewed changes
Copy link
Member Author

@massongit massongit left a comment

Choose a reason for hiding this comment

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

test

@massongit
Copy link
Member Author

test

This was referenced Jun 10, 2024
Closed
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Goryudyuma Goryudyuma Goryudyuma approved these changes

@nakkaa nakkaa Awaiting requested review from nakkaa

Assignees
No one assigned
Labels
review please
Projects
task list
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@massongit @Goryudyuma