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.

Sign up for GitHub

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

Jump to bottom

fix: Add SPDX license headers to all files and remove files without license information #352

Merged
merged 5 commits into from
Apr 26, 2024
Merged

fix: Add SPDX license headers to all files and remove files without license information #352

merged 5 commits into from
Apr 26, 2024

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Apr 25, 2024

  • Replace icons with unknown license with octicon icons
  • Update software icons with original icons and add license information
  • Add license headers to all of our files

@susnux susnux force-pushed the fix/add-license-headers branch from 9c3899d to 4d2b285 Compare April 25, 2024 14:13
@susnux
Copy link
Contributor Author

susnux commented Apr 25, 2024

Error
fsfe/reuse-action@a46482c is not allowed to be used in nextcloud/.github. Actions in this workflow must be: within a repository owned by nextcloud, created by GitHub, or matching the following: cirrus-actions/rebase@, ibiqlik/action-yamllint@, peter-evans/, skjnldsv/check-actor-permission@, xt0rted/block-autosquash-commits-action@, andstor/file-existence-action@, skjnldsv/block-fixup-merge-action@, webiny/action-conventional-commits@.

Seems like someone with permissions need to allow running the reuse workflow in this repository.

@susnux susnux added the 3. to review Waiting for reviews label Apr 25, 2024
@susnux susnux force-pushed the fix/add-license-headers branch from 4d2b285 to 5fb11ae Compare April 25, 2024 14:31
@AndyScherzinger
Copy link
Member

Seems like someone with permissions need to allow running the reuse workflow in this repository.

DONE 👍

@susnux susnux force-pushed the fix/add-license-headers branch from 5fb11ae to 2e18a9e Compare April 25, 2024 15:12
AndyScherzinger
AndyScherzinger approved these changes Apr 25, 2024
View reviewed changes
.github/workflows/reuse.yml Outdated Show resolved Hide resolved
susnux and others added 3 commits April 25, 2024 18:15
@susnux @AndyScherzinger
fix: Add SPDX license headers to all files and remove files without l…
a7b41dc 
…icense information

* Replace icons with unknown license with octicon icons
* Update software icons with original icons and add license information
* Add license headers to all of our files

Signed-off-by: Ferdinand Thiessen <[email protected]>
@AndyScherzinger
chore: Add missing dates
11f61ce 
Signed-off-by: Andy Scherzinger <[email protected]>
@AndyScherzinger
chore: remove push
9fc025e 
Signed-off-by: Andy Scherzinger <[email protected]>
@AndyScherzinger AndyScherzinger force-pushed the fix/add-license-headers branch from 49280a1 to 9fc025e Compare April 25, 2024 16:15
@AndyScherzinger
Copy link
Member

Rebased 🚀

@susnux susnux requested a review from nickvergessen April 26, 2024 14:32
@AndyScherzinger AndyScherzinger merged commit fc09713 into master Apr 26, 2024
5 checks passed
@AndyScherzinger AndyScherzinger deleted the fix/add-license-headers branch April 26, 2024 17:19
@skjnldsv
Copy link
Member

skjnldsv commented May 3, 2024

Could e have a description and logo please?
https://github.com/nextcloud/.github/tree/master/workflow-templates

@AndyScherzinger
Copy link
Member

DONE: #363

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

Assignees
No one assigned
Labels
3. to review Waiting for reviews technical debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@susnux @AndyScherzinger @skjnldsv @nickvergessen