-
Notifications
You must be signed in to change notification settings - Fork 0
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
QA-820 os licenses split - the frontend part #1
QA-820 os licenses split - the frontend part #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for your help! Just an observation: is it better, maybe, to use a single job for creating a single license file from both frontend and backend?
frontend/pipeline.yml
Outdated
paths: | ||
- licenses.md | ||
|
||
publish:frontend:licenses:docs-site: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe is it better to move this to the main .gitlab-ci.yml
and use a single publish:license:docs-site
for both backend and frontend?
Like (here)[https://github.com/mendersoftware/pull/220/files#diff-037ea159eb0a7cb0ac23b851e66bee30fb838ee8d0d99fa331a1ba65283d37f7R431] setting the two jobs:
needs:
- job: publish:backend:licenses
artifacts: true
- job: publish:frontend:licenses
artifacts: true
And get an unified LICENSE.md
file, also for a single mender-docs page?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure we can do that as well - I just carried on with the split until then, but I'll adjust 👌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Ticket: QA-820 Changelog: None Signed-off-by: Manuel Zedel <manuel.zedel@northern.tech>
3bf5a3f
to
6c84dc5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Thanks for your help!
chore: add depenabot configuration
The go-licenses format is not 100% matched, as I don't get the location of the license file from the license collection tool I get the
licenses.json
from - so this just links to the repository... I hope that's tolerable @oldgiova