[Bug 1974377] Review Request: rust-clang-ast - Data structures for processing Clang's `-ast-dump=json` format

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #3 from Jan Staněk <jstanek@xxxxxxxxxx> ---
(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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux