Skip to content
This repository was archived by the owner on Nov 11, 2019. It is now read-only.

Conversation

@La0
Copy link
Contributor

@La0 La0 commented May 24, 2019

No description provided.

@La0 La0 self-assigned this May 24, 2019
@La0 La0 force-pushed the move-phabricator branch from 32dec08 to 234bd96 Compare May 24, 2019 15:49
@La0 La0 marked this pull request as ready for review May 24, 2019 15:49
@La0 La0 requested a review from garbas as a code owner May 24, 2019 15:49
@La0 La0 requested a review from marco-c May 24, 2019 15:50
@La0
Copy link
Contributor Author

La0 commented May 24, 2019

Depends on mozilla/libmozdata#162

After a new build with git revision in nix packages, i'll release libmozdata on Pypi, then update the final requirements.

@La0
Copy link
Contributor Author

La0 commented May 24, 2019

@marco-c I released a new version of libmozdata, and use it here... Should be good to go if the build is green !

# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.

Copy link
Collaborator

Choose a reason for hiding this comment

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

We could actually remove the Phabricator endpoint in codecoverage/backend, #1992.

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 did a PR for that, and rebased this one on top (first 2 commits): no more src/codecoverage/backend changes here.

Copy link
Collaborator

@marco-c marco-c left a comment

Choose a reason for hiding this comment

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

Looks good. The changes in the backend could be avoided if we remove the Phabricator endpoint first (#1992).

@La0 La0 force-pushed the move-phabricator branch from 4cabe97 to 31c76cc Compare May 27, 2019 08:27
@La0
Copy link
Contributor Author

La0 commented May 27, 2019

This will need a rebase once #2132 lands

@La0 La0 force-pushed the move-phabricator branch from 31c76cc to 87877d2 Compare May 27, 2019 09:16
@La0 La0 merged commit 742fc2f into mozilla:master May 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants