https://bugzilla.redhat.com/show_bug.cgi?id=1974377



--- Comment #3 from Jan StanΔ›k <jsta...@redhat.com> ---
(In reply to Fabio Valentini from comment #2)

> 0) remove markdown markup from Summary
βœ”

> 1) "# check has missing dependencies"
listed clang-ast-test-suite βœ”

> 2) %description
> Looks like you changed "Clang's" to "Clang" in the generated description.
> Why?

It messes my syntax higlighting in vim, and I forgot to undo the change once I
was done with the spec. Apostrophe added back πŸ™‚

> 3) %license in %files

Changed as requested. βœ”

> 4) changelog date format

Time and timezone removed. βœ”

---

Thanks for the review! The one in kind should be approved now.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to