Ubuntu Calculator App

Merge lp://qastaging/~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-start-time-optimizations into lp://qastaging/ubuntu-calculator-app

  1. ubuntu-calculator-app-start-time-optimizations
  2. Merge into trunk
Proposed by Bartosz Kosiorek
Status: Merged
Approved by: Bartosz Kosiorek
Approved revision: 246
Merged at revision: 253
Proposed branch: lp://qastaging/~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-start-time-optimizations
Merge into: lp://qastaging/ubuntu-calculator-app
Diff against target: 82 lines (+16/-27)
1 file modified
app/ubuntu-calculator-app.qml (+16/-27)
To merge this branch: bzr merge lp://qastaging/~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-start-time-optimizations
Reviewer Review Type Date Requested Status
Jenkins Bot continuous-integration Approve
Riccardo Padovani Needs Fixing
Review via email: mp+278877@code.qastaging.launchpad.net

Commit message

Reduce start time by applying optimization (LP: #1520554)

Description of the change

Reduce start time by applying optimization (LP: #1520554)

To post a comment you must log in.
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :

PASSED: Continuous integration, rev:245
https://core-apps-jenkins.ubuntu.com/job/calculator-app-ci/31/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/193/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/calculator-app-ci/31/rebuild

review: Approve (continuous-integration)
Revision history for this message
Riccardo Padovani (rpadovani) wrote :

Hi Bartosz,
thanks for the branch, I like it.

Anyway, I don't think we want to add the "networking" policy to the release in production, while is useful in debug - so please drop it :)

review: Needs Fixing
lp://qastaging/~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-start-time-optimizations updated
246. By Bartosz Kosiorek

Remove not needed policy group

Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :

PASSED: Continuous integration, rev:246
https://core-apps-jenkins.ubuntu.com/job/calculator-app-ci/38/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/243/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/calculator-app-ci/38/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches