Closed Bug 1063473 Opened 10 years ago Closed 9 years ago

Grunt build should clean up .tmp/ directory once complete

Categories

(Tree Management :: Treeherder, defect, P5)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley, Mentored)

Details

(Whiteboard: [good first bug][lang=js])

User Story

Thank you for helping out with Treeherder!

You can find us on IRC at irc://irc.mozilla.org/treeherder

Here's some links to help get you started.

Project page:
https://wiki.mozilla.org/Auto-tools/Projects/Treeherder

Interacting with us, repo locations and links to set up a development version of the software:
https://wiki.mozilla.org/Auto-tools/Projects/Treeherder#Contributing
https://wiki.mozilla.org/Auto-tools/Projects/Treeherder#Source_and_Docs

A-Team general reference, coding style guides:
http://ateam-bootcamp.readthedocs.org

Attachments

(1 file)

Running grunt build leaves behind:
.tmp/concat/css/help.min.css
.tmp/concat/css/index.min.css
.tmp/concat/css/logviewer.min.css
.tmp/concat/js/index.min.js
.tmp/concat/js/logviewer.min.js

.tmp/ is listed in .gitignore, but it would still be nice to clean these up at the end of grunt build.
Blocks: 1072681
No longer blocks: 1072681
Component: Treeherder → Treeherder: Docs & Development
Priority: P4 → P5
Mentor: emorley
User Story: (updated)
Whiteboard: [good first bug]
Whiteboard: [good first bug] → [good first bug][lang=js]
User Story: (updated)
Assignee: nobody → emorley
Status: NEW → ASSIGNED
Attachment #8613293 - Flags: review?(cdawson)
Attachment #8613293 - Flags: review?(cdawson) → review+
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/6b89d96eced5c8a15781bea07a6ef230948f3a56
Bug 1063473 - Make grunt build clean up the .tmp directory

grunt-usemin uses .tmp/ as a staging area, however doesn't clean up
after itself. As such, let's run an additional grunt-clean step at the
end of the build, which removes this temporary staging directory.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Component: Treeherder: Docs & Development → TreeHerder
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: