This repository has been archived by the owner on Oct 15, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 123
[new-backend] spec hook #4495
Merged
atmaxinger
merged 11 commits into
ElektraInitiative:new-backend
from
atmaxinger:nb-hooks-spec
Sep 30, 2022
Merged
[new-backend] spec hook #4495
Changes from 6 commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
ae60eb6
hooks: initial implementation of spec hook and rewrite of spec plugin
atmaxinger 317ae67
hooks: simplify hook function names
atmaxinger d352f56
Restyled by clang-format
restyled-commits 47d1ae9
spec: implement basic remove hook
atmaxinger 2308f9b
spec: add test that it doesn't remove non-spec related meta
atmaxinger 73f17d7
Restyled by clang-format
restyled-commits edde04c
spec: add shell test to see if type plugin is ever called
atmaxinger 9d2b744
spec: commit real test
atmaxinger e123993
hooks: update discussion link for enabling/disabling hooks
atmaxinger a40c9c8
hooks/spec: update existing tests
atmaxinger 48f21e0
Restyled by prettier-markdown
restyled-commits File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I don't think we need to solve this before merging this PR. Can be added later at any time.
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.
Agreed.