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.

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 project structure #490

Merged
merged 2 commits into from
Apr 2, 2024
Merged

Conversation

afritzler
Copy link
Member

Proposed Changes

Refactore the current project structure to reflect the current conventions in Go. Create top-level packages for api, cmd and keep the implementation details in the internal package.

Also bump golang version to 1.22 and update the golangci-lint to v1.57.0.

@afritzler afritzler requested a review from a team as a code owner March 20, 2024 10:05
@github-actions github-actions bot added size/XL enhancement New feature or request labels Mar 20, 2024
@afritzler afritzler requested a review from lukasfrank March 20, 2024 10:06
@afritzler afritzler force-pushed the enh/refactore-project-structure branch 3 times, most recently from 846e636 to 1043e45 Compare March 20, 2024 10:14
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Mar 20, 2024
Refactore the current project structure to reflect the current
conventions in Go. Create top-level packages for `api`, `cmd` and keep
the implementation details in the `internal` package.

Also bump golang version to 1.22 and update the `golangci-lint` to
v1.57.0.
@afritzler afritzler force-pushed the enh/refactore-project-structure branch from 1043e45 to 58e2dd6 Compare March 20, 2024 10:20
@lukasfrank lukasfrank merged commit b44ff8c into main Apr 2, 2024
10 checks passed
@lukasfrank lukasfrank deleted the enh/refactore-project-structure branch April 2, 2024 06:44
lukas016 pushed a commit to opensovereigncloud/ceph-provider that referenced this pull request May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request integration-tests size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants