1
72%
master: 72%

Ran 14 Sep 2023 08:44AM UTC

Files 481

Run time 10s

Badge

Embed ▾

Committed 14 Sep 2023 08:39AM UTC coverage: 81.007% (+0.003%) from 81.004%

Job # 6183021731.1

Build Type

push

github

Committed by

web-flow

Commit Message

refactor: rename geolocate enginelocate (#1263)

I am soon going to start addressing the underlying issue described by
https://github.com/ooni/probe/issues/2531.

But, before doing that, I have noticed that the packages I need to edit
to this end are the following:

```
./internal/engine
./internal/geolocate
./internal/sessionresolver
```

Now, these three packages work in unison to provide an `engine.Session`
and they should *sort* together. However, I would like also to avoid
nesting because I think all of them servers an ~independent purpose and
geolocate and sessionresolver are possibly building blocks to refactor
or reimplement the engine.

For this reason, I have chosen to rename them such that it is clear they
are the engine and supporting packages.

This diff addresses the first half of the change by renaming the
geolocate package.

While there acknowledge that the script to rename packages was broken
and decide to ditch it for good rather than entering into the quest of
fixing it. I would probably have spent lots of time trying in doing
that, and my time is better spent otherwise. (I remember the official
package renaming tool did not support go modules, but probably it does
and I should look into giving it another spin.)

Run Details

22328 of 27563 relevant lines covered (81.01%)

39.45 hits per line

ooni / probe-cli / 6183021731 / 1
72%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 0 - 6183021731.1

ooni / probe-cli / 6183021731 / 1 72% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 0 - 6183021731.1

ooni / probe-cli / 6183021731 / 1
72%
master: 72%

README BADGES
x