-
Notifications
You must be signed in to change notification settings - Fork 16
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
chore: Add SPDX headers #1392
chore: Add SPDX headers #1392
Conversation
Signed-off-by: Jonas <[email protected]>
6bd4842
to
12413d7
Compare
3 failed and 2 flaky tests on run #1744 ↗︎
Details:
pages.spec.js • 1 failed test • Nextcloud master
page-details.spec.js • 1 failed test • Nextcloud master
page-landingpage.spec.js • 1 failed test • Nextcloud stable27
page-list.spec.js • 1 flaky test • Nextcloud stable27
page-details.spec.js • 1 flaky test • Nextcloud stable27
Review all test suite changes for PR #1392 ↗︎ |
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 work @mejo-
The remaining items are mostly to be tackled via .license
files and a REUSE.toml
Can be done via follow-up PR. I am happy to do thin in my evening hobby time 😃
Other than that: Looks good 👍
6ed4181
to
5592a83
Compare
Fixes: #1367 Signed-off-by: Jonas <[email protected]>
Thanks @AndyScherzinger 🙏 I made the reuse CI job pass now, so hope I caught all the remainers. Still follow-up PRs are much appreciated if you still spot anything 😊 |
🏁 Checklist
npm run lint
/npm run stylelint
/composer run cs:check
)