code-review

code-review copied to clipboard

It can be seen in [D187545](https://phabricator.services.mozilla.com/D187545) that there were no tasks associated with this revision. Looking other the logs there was a local mercurial error even if the push was...

abpostelnicu

Our pkeys in our db use django sequences with pgsql, this works fine if you have a fairly small number of rows but since we populate on a daily bases...

abpostelnicu

clang-format linter has some rendering defects

2

It can be seen in [D181494](https://phabricator.services.mozilla.com/D181494#C6574269NL55) that the second `for loop` has nothing to do in the current context and must be associated with line 248. This is a rendering...

abpostelnicu

In PR #1804 we kept back sentry-sdk 1.22.2 because later versions break our integration, we should mitigate this with tests.

abpostelnicu

Reviewbot adding #taskgraph-reviewers to all patches in a stack that modify graph

7

In https://github.com/mozilla/code-review/issues/1475 a rule was added to automatically `#taskgraph-reviewers` to a revision if the tgdiff task detects changes to the task graph. In https://phabricator.services.mozilla.com/D177718, it correctly added `#taskgraph-reviewers`. But then...

ahal

Bumps [vue-chartist](https://github.com/Yopadd/vue-chartist) from 2.3.1 to 3.0.0. Release notes Sourced from vue-chartist's releases. v3.0.0 NEWS vue-chartist is ready for vuejs3 Commits See full diff in compare view [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) You...

dependabot[bot]

Get repository and revision from code-review task preferentially.

3

tomprince

Fixes #31 Refs #25

La0

Re-implement before/after in the new infrastructure

1

This is a requirement for #32.

marco-c

With latest revision by date, in case the parent revision of a patch is not public or was not analyzed and stored in the backend for some reason, we can...

marco-c