Hook up `npm version` to be able to bump the install.rdf.in version number and be ready to tag releases

RESOLVED FIXED

Status

Hello (Loop)
Client
P1
normal
Rank:
10
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
I discovered that we can use `npm version` to update the version number in package.json. Additionally, it can also run pre, during & post scripts to do additional tasks.

Hence, to begin with I want to:

- Pre-version bump: run tests (sanity check)
- During version bump: change the version in install.rdf.in.
- Hook up enough to bump the add-on version on for master.
- Start adding some release documentation
Created attachment 8710704 [details] [review]
[loop] Standard8:bug-1241649-ver > mozilla:master
(Assignee)

Comment 2

2 years ago
Comment on attachment 8710704 [details] [review]
[loop] Standard8:bug-1241649-ver > mozilla:master

Fairly small PR to make it easy for us to bump version numbers.

Includes the start of release documentation.

I've also included an example commit of bumping the version numbers that I want to push anyway, as that will match the developer version of the add-on that I've just put on AMO.
Attachment #8710704 - Flags: review?(edilee)
Attachment #8710704 - Flags: review?(dmose)
Attachment #8710704 - Flags: review?(dcritchley)

Comment 3

2 years ago
Comment on attachment 8710704 [details] [review]
[loop] Standard8:bug-1241649-ver > mozilla:master

r=Mardak with some small script suggestions
Attachment #8710704 - Flags: review?(edilee)
Attachment #8710704 - Flags: review?(dmose)
Attachment #8710704 - Flags: review?(dcritchley)
Attachment #8710704 - Flags: review+
(Assignee)

Comment 4

2 years ago
https://github.com/mozilla/loop/commit/6054f48985b30af98446ccd4b08a06aefc071bd4
https://github.com/mozilla/loop/commit/177631d2695599d1d90cd6a3b28412f146cc8d23
Status: NEW → RESOLVED
Iteration: --- → 46.3 - Jan 25
Last Resolved: 2 years ago
Rank: 10
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.