-
Notifications
You must be signed in to change notification settings - Fork 22
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
Package restructuring #784
Conversation
This PR exceeds the recommended size of 10000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Mend Scan Summary: ❌Repository: open-component-model/ocm
|
Integration Tests for 862458c run with result: Success ✅! |
This PR exceeds the recommended size of 10000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Integration Tests for 49a79ea run with result: Success ✅! |
This PR exceeds the recommended size of 10000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Integration Tests for 49a79ea run with result: Success ✅! |
00955c0
to
1cbbc16
Compare
Integration Tests for 1ba6e56 run with result: Success ✅! |
Integration Tests for e76ec98 run with result: Success ✅! |
This PR exceeds the recommended size of 10000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Integration Tests for 02592a6 run with result: Success ✅! |
This PR exceeds the recommended size of 10000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Integration Tests for 5830454 run with result: Success ✅! |
This PR exceeds the recommended size of 10000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Integration Tests for 5830454 run with result: Success ✅! |
Integration Tests for d117e69 run with result: Success ✅! |
5994168
to
9edec24
Compare
Integration Tests for d117e69 run with result: Success ✅! |
Integration Tests for d117e69 run with result: Success ✅! |
Integration Tests for 98e3650 run with result: Success ✅! |
Integration Tests for 8472909 run with result: Success ✅! |
Description
Restructure the ocm core library according to the intended new package structure and module name (
ocm.software/ocm
).In a first step we keep the lib and the CLI inthe same repository as before.
The actual first migration step has been be done based on the original main branch just
by calling
migrate.sh --paths --module
from the repository root folder. It can be repeated for new main versions.Afterwards it is possible to call
make build
make generate
make test
make generate-deepcopy
go fmt ./{api,cmds}/...
make check
These commands should succeed.
This is the current state. Everything else is not yet done.
So far, packages are moved and renamed to their intended location in the package structure.
The used relative paths should be adapted and tests and component builds are running.
To adapt projects using this library a migration script is provided.
It can be run in in the root folder of using repositories by calling
migrate.sh --script migrate.mig
.Optionally, particular package folders can be given as additional arguments.
(see also mig.md)
What type of PR is this? (check all applicable)
Related Tickets & Documents