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

Jump to bottom

restructure code tree #518

Closed
pashagolub opened this issue Aug 25, 2024 · 0 comments · Fixed by #521
Closed

restructure code tree #518

pashagolub opened this issue Aug 25, 2024 · 0 comments · Fixed by #521
Assignees
@pashagolub
Labels
refactoring Something done as it should've been done from the start

Comments

@pashagolub
Copy link
Collaborator

  • get rid of src folder
  • put go.mod to the root folder
  • put most packages in internal
  • provide one public package to allow import for rpc sinks implementations
@pashagolub pashagolub self-assigned this Aug 25, 2024
@pashagolub pashagolub added the refactoring Something done as it should've been done from the start label Aug 25, 2024
pashagolub added a commit that referenced this issue Aug 26, 2024
@pashagolub
[!] restructure code tree, closes #518
47b43c8 
- get rid of `src` folder
- put `go.mod` to the root folder
- put packages in `internal`
- provide one public `api` package to allow import for rpc sinks
- put `main.go` under `cmd\pgwatch`
- move `utils\convert` to `cmd\pgwatch`
@pashagolub pashagolub linked a pull request Aug 26, 2024 that will close this issue
[!] restructure code tree, closes #518 #521
Merged
pashagolub added a commit that referenced this issue Aug 26, 2024
@pashagolub
[!] restructure code tree, closes #518 (#521)
364061e 
* [!] restructure code tree, closes #518
- get rid of `src` folder
- put `go.mod` to the root folder
- put packages in `internal`
- provide one public `api` package to allow import for rpc sinks
- put `main.go` under `cmd\pgwatch`
- move `utils\convert` to `cmd\pgwatch`
* [*] rename `MeasurementMessage` to `MeasurementEnvelope`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pashagolub pashagolub

Labels
refactoring Something done as it should've been done from the start
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[!] restructure code tree, closes #518 cybertec-postgresql/pgwatch
1 participant
@pashagolub