Bug 1749751 - Review Request: rubygem-kramdown-parser-gfm - Kramdown parser for GitHub-flavored markdown
Summary: Review Request: rubygem-kramdown-parser-gfm - Kramdown parser for GitHub-flav...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1749749
Blocks: 1743745
TreeView+ depends on / blocked
 
Reported: 2019-09-06 11:23 UTC by Fabio Valentini
Modified: 2019-09-13 13:27 UTC (History)
1 user (show)

Fixed In Version: rubygem-kramdown-parser-gfm-1.1.0-1.fc32
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-09-13 13:27:57 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+


Attachments (Terms of Use)

Description Fabio Valentini 2019-09-06 11:23:20 UTC
Spec URL: https://decathorpe.fedorapeople.org/packages/rubygem-kramdown-parser-gfm.spec
SRPM URL: https://decathorpe.fedorapeople.org/packages/rubygem-kramdown-parser-gfm-1.1.0-1.fc30.src.rpm

Description:
kramdown-parser-gfm provides a kramdown parser for the GFM dialect of
Markdown.

Fedora Account System Username: decathorpe


COPR repo with working packages:
https://copr.fedorainfracloud.org/coprs/decathorpe/jekyll4/monitor/

Comment 1 Mamoru TASAKA 2019-09-13 03:18:57 UTC
1. Review

* license okay
  - MIT for all files
* spec file 
* mock build okay
  https://koji.fedoraproject.org/koji/taskinfo?taskID=37637054
* rpmlint okay for src.rpm, binary rpm, installed rpm
* installable for F-32
* basic functionality okay (as documentated on "Usage" of https://github.com/kramdown/parser-gfm)
```
[mockbuild@1a996dddfd3e4efb85d341420b9e76a2 ~]$ ruby -e 'require "kramdown" ; require "kramdown-parser-gfm" ; text="This ~~is a complex strike through *test ~~with nesting~~ involved* here~~." ; puts Kramdown::Document.new(text, input: "GFM").to_html'
<p>This <del>is a complex strike through *test ~~with nesting</del> involved* here~~.</p>
```

2. Advice
* BR
  - I would recommend to write "BuildRequires:  rubygem(kramdown) >= 2.0" because on F-31 and below rubygem-kramdown is still 1.7.x and
    this srpm is incompatible on those branches (not a blocker for review)

By the way do you plan to import this also on F-31? If so I will coordinate it.

Comment 2 Mamoru TASAKA 2019-09-13 03:19:39 UTC
============================================================
  This package (rubygem-kramdown-parser-gfm) is APPROVED
  by mtasaka
============================================================

Comment 3 Fabio Valentini 2019-09-13 11:06:51 UTC
Thanks for the review!

BuildRequires:  rubygem(kramdown) >= 2.0.0
is already in the .spec file. ;)

I only intend to push this to fedora 32, since it's a breaking change (in both kramdown and jekyll 4).
See: https://fedoraproject.org/wiki/Changes/Jekyll4

Comment 5 Gwyn Ciesla 2019-09-13 13:07:07 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rubygem-kramdown-parser-gfm

Comment 6 Fabio Valentini 2019-09-13 13:27:57 UTC
Built for rawhide.
https://koji.fedoraproject.org/koji/buildinfo?buildID=1377189


Note You need to log in before you can comment on or make changes to this bug.