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

Add summary and install time properties from the RPM database Generat… #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NiuBlibing
Copy link

…e updated test cases using a shell script

  • add install time property
  • add summary property
  • add a shell to generate test cases
  • TODO: resolve multilanguage summary properity

Signed-off-by: Tim Ferguson [email protected]
Signed-off-by: Tianyue Ren [email protected]

Issue 16

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

…e updated test cases using a shell script

- add install time property
- add summary property
- add a shell to generate test cases
- TODO: resolve multilanguage summary properity

Signed-off-by: Tim Ferguson <[email protected]>
Signed-off-by: Tianyue Ren <[email protected]>
@bcrochet
Copy link
Contributor

+1 to this. Was going to submit the summary part myself.

Copy link
Owner

@knqyf263 knqyf263 left a comment

Choose a reason for hiding this comment

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

Looks like your change includes CR (carriage return) and it makes differences a lot. Could you fix it?

@bcrochet
Copy link
Contributor

bcrochet commented Apr 4, 2022

@NiuBlibing Can you remove the CRs from the PR as @knqyf263 has requested?

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