Skip to content
This repository has been archived by the owner on Feb 2, 2019. It is now read-only.

Specify build properties in extra #144

Merged
merged 3 commits into from
Mar 11, 2016
Merged

Specify build properties in extra #144

merged 3 commits into from
Mar 11, 2016

Conversation

rail
Copy link
Contributor

@rail rail commented Mar 10, 2016

locales: ["en-US"]
branch: {{ branch }}
platform: {{ platform }}
version: "{{ version }}"
Copy link

Choose a reason for hiding this comment

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

Do you need the quotes here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah. We hit a bug when 45.0 was interpreted as 45.

Copy link
Contributor

Choose a reason for hiding this comment

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

should we stick product in here now to prep for upcoming fennec or will we have logic against platform?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think we will revisit this multiple times :)

@rail
Copy link
Contributor Author

rail commented Mar 10, 2016

@lundjordan ?

@lundjordan
Copy link
Contributor

🚀 it

rail pushed a commit that referenced this pull request Mar 11, 2016
Specify build properties in extra
@rail rail merged commit dc039f3 into mozilla-releng:master Mar 11, 2016
@rail rail deleted the mozmill branch March 14, 2016 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants