Skip to content
New issue

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

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

Already on GitHub? # to your account

Merge dev into main: Update Golang version and logging system #438

Merged
merged 2 commits into from
Oct 22, 2024
Merged

Conversation

moonD4rk
Copy link
Owner

@moonD4rk moonD4rk commented Oct 22, 2024

Proposed changes

This PR merges recent changes from the dev branch into the main branch. Key updates include

  • Downgrading Golang version to 1.20 to support Windows 7.
  • Replacing log/slog with the standard logger for improved maintainability and performance

Checklist

  • Pull request is created against the dev branch
  • All checks passed (lint, unit, build tests etc.) with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

* chore: downgrade golang version to 1.20, support windows 7

* chore: Update dependencies for Go project.

- Update dependencies in go.sum
- Improvements and optimizations in various files
- Bug fixes and error handling enhancements

* chore: Update modernc.org/sqlite library versions in go.mod and go.sum files

- Update version of `modernc.org/sqlite` to `v1.31.1` in `go.mod` and `go.sum` files
- Update module hash in `go.sum` file for `modernc.org/sqlite`
- Ensure consistency between `go.mod` and `go.sum` files in relation to `modernc.org/sqlite` version

* chore: replace log/slog with standard logger (#436)

* chore: replace log/slog with standard logger

* chore: Update Go dependencies and versions

- Update Go version from `1.22.5` to `1.20` and other dependencies
- Update critical dependencies to latest versions
- Ensure compatibility with new versions of dependencies

* chore: Optimize dependency management in workflows

- Update build and lint workflows to use `go mod tidy` for getting dependencies
- Change modules download mode to `'mod'` in linters configuration
- Add step to get dependencies in lint workflow

* refactor: Update dependencies and refactor Chromium key deletion logic

- Update `modernc.org/sqlite` to `v1.31.1` in `go.mod` and `go.sum`
- Increase version number to `0.5.0` in `cmd/hack-browser-data/main.go`
- Refactor and update logic for filtering and copying items in `browser/chromium/chromium.go`

* Improve logging functionality and data type conversion

- Add `String()` method to `DataType` enum in types.go
- Update log level to Debug in logger_test.go
- Set log level to Debug in `TestLoggerDebug` and `TestLoggerDebugf` functions
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11452927924

Details

  • 91 of 182 (50.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+23.6%) to 42.273%

Changes Missing Coverage Covered Lines Changed/Added Lines %
browser/firefox/firefox.go 0 1 0.0%
browserdata/localstorage/localstorage.go 0 2 0.0%
browserdata/browserdata.go 0 6 0.0%
log/logger.go 83 98 84.69%
log/log.go 0 27 0.0%
types/types.go 8 48 16.67%
Totals Coverage Status
Change from base Build 11105027399: 23.6%
Covered Lines: 372
Relevant Lines: 880

💛 - Coveralls

@moonD4rk moonD4rk merged commit ad020cf into main Oct 22, 2024
8 checks passed
@moonD4rk moonD4rk added this to the v0.5.0 milestone Oct 22, 2024
@moonD4rk moonD4rk deleted the dev branch October 28, 2024 10:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants