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.

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

fix: refactor names - RK-18987 #51

Merged
merged 1 commit into from
Jun 22, 2023
Merged

fix: refactor names - RK-18987 #51

merged 1 commit into from
Jun 22, 2023

Conversation

EliRookout
Copy link
Contributor

@EliRookout EliRookout commented Jun 20, 2023

Signed-off-by: Eli Hatamov [email protected]

Pull Request

Description

Please provide a brief description of the changes made in this pull request.

Related Issue(s)

If this pull request addresses or relates to any open issues, please mention them here using the syntax Fixes #<issue_number> or Resolves #<issue_number>.

Checklist

Before submitting this pull request, please ensure that you have completed the following tasks:

Testing Instructions

Please provide clear instructions on how to test and verify the changes made in this pull request.

Additional Information

Add any additional information or context that would be helpful in understanding and reviewing this pull request.

Signed-off-by: Eli Hatamov <[email protected]>
@gosharo
Copy link
Contributor

gosharo commented Jun 20, 2023

Github Enforcer opened Task: RK-18987

@sonariorobot sonariorobot changed the title fix: refactor names RK-18987 - fix: refactor names Jun 20, 2023
@gosharo
Copy link
Contributor

gosharo commented Jun 20, 2023

LGTM, @nissim-infra ?

@gosharo gosharo changed the title RK-18987 - fix: refactor names fix: refactor names - RK-18987 Jun 20, 2023
@gosharo gosharo merged commit 36e066f into main Jun 22, 2023
@gosharo gosharo deleted the refactor-names branch June 22, 2023 15:14
gosharo pushed a commit that referenced this pull request Jun 22, 2023
Signed-off-by: Eli Hatamov <[email protected]>
gosharo pushed a commit that referenced this pull request Jun 22, 2023
Signed-off-by: Eli Hatamov <[email protected]>
Signed-off-by: Gosha <[email protected]>
gosharo pushed a commit that referenced this pull request Jun 22, 2023
Signed-off-by: Eli Hatamov <[email protected]>
gosharo pushed a commit that referenced this pull request Jun 22, 2023
Signed-off-by: Eli Hatamov <[email protected]>
Signed-off-by: Gosha <[email protected]>
gosharo added a commit that referenced this pull request Jun 23, 2023
* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

* docs: minor fixes

Signed-off-by: Gosha <[email protected]>

* docs: add missing Installation file

Signed-off-by: Gosha <[email protected]>

* fix: refactor names (#51)

Signed-off-by: Eli Hatamov <[email protected]>
Signed-off-by: Gosha <[email protected]>

* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

* fix: config injection new config validation

Signed-off-by: Gosha <[email protected]>

---------

Signed-off-by: Gosha <[email protected]>
Signed-off-by: Eli Hatamov <[email protected]>
Signed-off-by: GoshaDozoretz <[email protected]>
Co-authored-by: Eli Hatamov <[email protected]>
gosharo added a commit that referenced this pull request Jun 27, 2023
* fix: git tests added

Signed-off-by: Gosha <[email protected]>

* fix: git tests added

Signed-off-by: Gosha <[email protected]>

* docs: minor fixes

Signed-off-by: Gosha <[email protected]>

* docs: add missing Installation file

Signed-off-by: Gosha <[email protected]>

* fix: refactor names (#51)

Signed-off-by: Eli Hatamov <[email protected]>
Signed-off-by: Gosha <[email protected]>

* fix: blank config map support (#52)

Signed-off-by: Gosha <[email protected]>

* fix: git test added

Signed-off-by: Gosha <[email protected]>

* fix: git test added

Signed-off-by: Gosha <[email protected]>

* add string pointer util

Signed-off-by: Gosha <[email protected]>

* save

Signed-off-by: Gosha <[email protected]>

* nir

Signed-off-by: Gosha <[email protected]>

---------

Signed-off-by: Gosha <[email protected]>
Signed-off-by: Eli Hatamov <[email protected]>
Signed-off-by: GoshaDozoretz <[email protected]>
Co-authored-by: Eli Hatamov <[email protected]>
Co-authored-by: Adi Ludmer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants