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

move migrate tooling to 'hack' folder #889

Merged
merged 2 commits into from
Aug 23, 2024
Merged

move migrate tooling to 'hack' folder #889

merged 2 commits into from
Aug 23, 2024

Conversation

hilmarf
Copy link
Member

@hilmarf hilmarf commented Aug 23, 2024

What this PR does / why we need it

cleanup root folder

Which issue(s) this PR fixes

related to open-component-model/ocm-project#243

@hilmarf hilmarf added this to the 2024-Q3 milestone Aug 23, 2024
@hilmarf hilmarf linked an issue Aug 23, 2024 that may be closed by this pull request
5 tasks
Copy link
Contributor

github-actions bot commented Aug 23, 2024

Mend Scan Summary: ❌

Repository: open-component-model/ocm

VIOLATION DESCRIPTION NUMBER OF VIOLATIONS
HIGH/CRITICAL SECURITY VULNERABILITIES 4
MAJOR UPDATES AVAILABLE 0
LICENSE REQUIRES REVIEW 3
HIGH RISK LICENSES 10
RESTRICTIED LICENSE FOR ON-PREMISE DELIVERY 0

Detailed Logs: mend-scan-> Generate Report
Mend UI

@hilmarf hilmarf enabled auto-merge (squash) August 23, 2024 10:48
@hilmarf hilmarf requested review from a team, frewilhelm, morri-son and morrison-sap August 23, 2024 10:48
Copy link
Contributor

ocmbot bot commented Aug 23, 2024

Integration Tests for f2dec73 run with result: Success ✅!

@morri-son
Copy link
Contributor

Hi @hilmarf , rödelt das Skript nicht auf Verzeichnissen rum, z.B.

# ocm extensions
for s in api/ocm/{repositories,accessmethods,blobhandler,attrs,download,actionhandler,digester,labels,resourcetypes,pubsub}; do
  mig $s api/ocm/extensions "${build[@]}"
done

Da müsste man eventuell das Skript vorher noch mit cd .. wieder an den richtigen Einstiegspunkt setzen, oder?

@hilmarf
Copy link
Member Author

hilmarf commented Aug 23, 2024

rödelt das Skript nicht auf Verzeichnissen rum, z.B.

true - but since we have done the migration already, it's not used in this repository anyhow/anymore. Still we should keep it, because it can easily be adapted to migrate other ocm-lib consumer repos.

@hilmarf hilmarf merged commit 48ee410 into main Aug 23, 2024
19 checks passed
@hilmarf hilmarf deleted the mv/migrate branch August 23, 2024 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔒Closed
Development

Successfully merging this pull request may close these issues.

Tidy root directory of ocm-repository
2 participants