Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: migrate bookmark static pages to new http server #775

Merged
merged 27 commits into from
Dec 28, 2023

added tests, refactor storage to use afero

f1a1305
Select commit
Loading
Failed to load commit list.
Merged

refactor: migrate bookmark static pages to new http server #775

added tests, refactor storage to use afero
f1a1305
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 25, 2023 in 1s

56.83% of diff hit (target 22.06%)

View this Pull Request on Codecov

56.83% of diff hit (target 22.06%)

Annotations

Check warning on line 48 in internal/cmd/add.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/add.go#L48 

Added line #L48 was not covered by tests

Check warning on line 104 in internal/cmd/add.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/add.go#L104 

Added line #L104 was not covered by tests

Check warning on line 79 in internal/cmd/check.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/check.go#L79 

Added line #L79 was not covered by tests

Check warning on line 55 in internal/cmd/import.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/import.go#L55 

Added line #L55 was not covered by tests

Check warning on line 138 in internal/cmd/import.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/import.go#L138 

Added line #L138 was not covered by tests

Check warning on line 39 in internal/cmd/pocket.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/pocket.go#L39 

Added line #L39 was not covered by tests

Check warning on line 96 in internal/cmd/pocket.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/pocket.go#L96 

Added line #L96 was not covered by tests

Check warning on line 52 in internal/cmd/root.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/root.go#L52 

Added line #L52 was not covered by tests

Check warning on line 105 in internal/cmd/root.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/root.go#L101-L105 

Added lines #L101 - L105 were not covered by tests

Check warning on line 40 in internal/cmd/server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/server.go#L40 

Added line #L40 was not covered by tests

Check warning on line 72 in internal/cmd/server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/server.go#L69-L72 

Added lines #L69 - L72 were not covered by tests

Check warning on line 150 in internal/cmd/update.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/update.go#L150 

Added line #L150 was not covered by tests

Check warning on line 178 in internal/cmd/update.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/update.go#L178 

Added line #L178 was not covered by tests

Check warning on line 44 in internal/cmd/utils.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/utils.go#L44 

Added line #L44 was not covered by tests

Check warning on line 32 in internal/domains/accounts.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/accounts.go#L32 

Added line #L32 was not covered by tests

Check warning on line 52 in internal/domains/accounts.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/accounts.go#L52 

Added line #L52 was not covered by tests

Check warning on line 72 in internal/domains/accounts.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/accounts.go#L72 

Added line #L72 was not covered by tests

Check warning on line 74 in internal/domains/accounts.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/accounts.go#L74 

Added line #L74 was not covered by tests

Check warning on line 82 in internal/domains/accounts.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/accounts.go#L80-L82 

Added lines #L80 - L82 were not covered by tests

Check warning on line 17 in internal/domains/archiver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/archiver.go#L17 

Added line #L17 was not covered by tests

Check warning on line 24 in internal/domains/archiver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/archiver.go#L24 

Added line #L24 was not covered by tests

Check warning on line 30 in internal/domains/archiver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/archiver.go#L30 

Added line #L30 was not covered by tests

Check warning on line 41 in internal/domains/archiver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/archiver.go#L40-L41 

Added lines #L40 - L41 were not covered by tests

Check warning on line 44 in internal/domains/archiver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/archiver.go#L43-L44 

Added lines #L43 - L44 were not covered by tests

Check warning on line 48 in internal/domains/archiver.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/domains/archiver.go#L48 

Added line #L48 was not covered by tests
View more details on Codecov