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

Implemented changes to adapt routingkey template to sepia #247

Merged
merged 25 commits into from
Aug 22, 2023

Conversation

jainadc9
Copy link
Contributor

@jainadc9 jainadc9 commented Aug 4, 2022

Applicable Issues

Description of the Change

Implemented changes to adapt routingkey template to sepia

Alternate Designs

Benefits

Possible Drawbacks

Sign-off

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or

(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or

(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.

(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

Signed-off-by: JainadC jainad.chinta@ericsson.com

Copy link
Contributor

@z-sztrom z-sztrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, define constants instead of using strings in code.

@e-backmark-ericsson
Copy link
Member

@jainadc9 do you have any answers to the review comments on this PR?

jainad and others added 3 commits December 22, 2022 18:31
@jainadc9 jainadc9 requested a review from z-sztrom March 2, 2023 10:35
@jainadc9 jainadc9 requested a review from z-sztrom August 16, 2023 12:15
Copy link
Contributor

@z-sztrom z-sztrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Do we have a test case for routing key file?
  • There's a lot of occurences ...Routingkey... It should use uppercase k ('K') instead.

@jainadc9 jainadc9 requested a review from z-sztrom August 17, 2023 06:31
Copy link
Contributor

@z-sztrom z-sztrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine.

@e-backmark-ericsson
Copy link
Member

It looks good to me

@jainadc9 jainadc9 merged commit 7ade041 into eiffel-community:master Aug 22, 2023
1 check passed
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.

3 participants