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

Jump to bottom

Actually remove six from Dockerfile #24514

Merged
merged 1 commit into from Jul 9, 2020
Merged

Actually remove six from Dockerfile #24514

merged 1 commit into from Jul 9, 2020

Conversation

Hexcles
Copy link
Member

@Hexcles Hexcles commented Jul 8, 2020

With #24484 removing the dependency on system six, we are now able to
remove it from the Docker image, which was introduced in #23733.

Removing this has the benefit of avoiding introducing accidental
dependency.

@Hexcles
Actually remove six from Dockerfile
bd5e423 
With #24484 removing the dependency on system six, we are now able to
remove it from the Docker image, which was introduced in #23733.

Removing this has the benefit of avoiding introducing accidental
dependency.
@Hexcles Hexcles mentioned this pull request Jul 8, 2020
Merged
@Hexcles Hexcles closed this Jul 8, 2020
@Hexcles Hexcles reopened this Jul 8, 2020
@Hexcles Hexcles merged commit dea212b into master Jul 9, 2020
@Hexcles Hexcles deleted the remove-six branch July 9, 2020 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jgraham jgraham jgraham approved these changes

Assignees

@LukeZielinski LukeZielinski

Labels
ci docker infra Taskcluster
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Hexcles @jgraham @wpt-pr-bot @LukeZielinski