[Identity] Reorder Dockerfile contents
ClosedPublic
Actions

Authored by varun on Apr 22 2022, 2:17 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Oct 25, 12:06 AM

	Unknown Object (File)
	Fri, Oct 25, 12:06 AM

	Unknown Object (File)
	Fri, Oct 25, 12:05 AM

	Unknown Object (File)
	Fri, Oct 25, 12:05 AM

	Unknown Object (File)
	Sun, Oct 13, 9:19 PM

	Unknown Object (File)
	Sun, Oct 13, 9:19 PM

	Unknown Object (File)
	Sun, Oct 13, 9:19 PM

	Unknown Object (File)
	Sun, Oct 13, 9:16 PM

Subscribers

Details

Reviewers

• jim
atul

Commits

rCOMMe6e82dcdb360: [Identity] Reorder Dockerfile contents

Summary

Addressing a comment from @jimpo in a previous diff. We should install rustfmt immediately after setting the user.

Test Plan

Build the docker image

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Apr 22 2022, 2:17 PM

Herald added subscribers: • abosh, • karol, • adrian and 2 others. · View Herald TranscriptApr 22 2022, 2:17 PM

Harbormaster completed remote builds in B8473: Diff 11814.Apr 22 2022, 2:22 PM

varun requested review of this revision.Apr 22 2022, 2:22 PM

atul accepted this revision.Apr 22 2022, 2:27 PM

This revision is now accepted and ready to land.Apr 22 2022, 2:27 PM

Original comment is in D3593

Closed by commit rCOMMe6e82dcdb360: [Identity] Reorder Dockerfile contents (authored by varun). · Explain WhyApr 22 2022, 2:29 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMe6e82dcdb360: [Identity] Reorder Dockerfile contents.

Revision Contents
Changeset List

Path

Size

services/

identity/

Dockerfile

6 lines

Diff 11817

View Options

services/identity/Dockerfile

[Identity] Reorder Dockerfile contentsClosedPublicActions