[Identity] Reorder Dockerfile contents
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F3115423: D3818.id11817.diff
	Thu, Oct 31, 9:28 PM

	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

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

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

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 11814

View Options

services/identity/Dockerfile

[Identity] Reorder Dockerfile contentsClosedPublicActions