Skip to content

refactor: Tide the directories of modules#3649

Merged
mrCherry97 merged 4 commits intokyma-project:mainfrom
dbadura:modules-refactor-3
Feb 5, 2025
Merged

refactor: Tide the directories of modules#3649
mrCherry97 merged 4 commits intokyma-project:mainfrom
dbadura:modules-refactor-3

Conversation

@dbadura
Copy link
Contributor

@dbadura dbadura commented Jan 31, 2025

Description

Changes proposed in this pull request:

  • move Components to components directory
  • remove jsonata not used functions

Related issue(s)
Closes #3649

Definition of done

  • The PR's title starts with one of the following prefixes:
    • feat: A new feature
    • fix: A bug fix
    • docs: Documentation only changes
    • refactor: A code change that neither fixes a bug nor adds a feature
    • test: Adding tests
    • chore: Maintainance changes to the build process or auxiliary tools, libraries, workflows, etc.
  • Related issues are linked. To link internal trackers, use the issue IDs like backlog#4567
  • Explain clearly why you created the PR and what changes it introduces
  • All necessary steps are delivered, for example, tests, documentation, merging

@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 31, 2025
@kyma-bot kyma-bot added the lgtm Looks good to me! label Feb 5, 2025
@mrCherry97
Copy link
Contributor

/override tide

@kyma-bot
Copy link
Contributor

kyma-bot commented Feb 5, 2025

@mrCherry97: Overrode contexts on behalf of mrCherry97: tide

Details

In response to this:

/override tide

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@mrCherry97 mrCherry97 merged commit 7636c3c into kyma-project:main Feb 5, 2025
56 of 61 checks passed
@dbadura dbadura deleted the modules-refactor-3 branch February 5, 2025 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants