Skip to content
This repository has been archived by the owner on Jul 17, 2023. It is now read-only.

Add deprecation notice #6

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Add deprecation notice #6

merged 1 commit into from
Jul 17, 2023

Conversation

bastjan
Copy link
Contributor

@bastjan bastjan commented Jul 17, 2023 •
edited
Loading

Checklist

  • The PR has a meaningful title. It will be used to auto generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

@bastjan bastjan requested a review from a team as a code owner July 17, 2023 08:36
@bastjan bastjan self-assigned this Jul 17, 2023
@bastjan bastjan added the documentation Improvements or additions to documentation label Jul 17, 2023
@bastjan bastjan merged commit 2e529f2 into master Jul 17, 2023
6 checks passed
@bastjan bastjan deleted the bastjan-patch-1 branch July 17, 2023 08:42
@simu
Copy link
Member

simu commented Jul 17, 2023

Maybe consider also deprecating the component in class/defaults.yml, cf. e.g. https://github.com/projectsyn/component-resource-locker/blob/41946894da5302c2da92b8ae6a880855c6e9e55a/class/defaults.yml#L3-L5

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

@simu simu simu approved these changes

Assignees

@bastjan bastjan

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bastjan @simu