https://bugzilla.redhat.com/show_bug.cgi?id=1883732
Pavel Valena changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
Fixed In Version|
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #12 from Gwyn Ciesla ---
(fedscm-admin): The Pagure repository was created at
https://src.fedoraproject.org/rpms/rubygem-sassc-rails
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always not
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
Vít Ondruch changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #11 from Vít
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #10 from Pavel Valena ---
(In reply to Vít Ondruch from comment #9)
> Just to make it clear, I am reviewing this .spec file and SRPM:
Yes, that's the .spec file I was pointing to, for review.
> Obviously, we still have rubyg
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #9 from Vít Ondruch ---
Just to make it clear, I am reviewing this .spec file and SRPM:
https://download.copr.fedorainfracloud.org/results/pvalena/rubygems/fedora-rawhide-x86_64/01722749-rubygem-sassc-rails/rubygem-sassc-rails.spe
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
Vít Ondruch changed:
What|Removed |Added
Flags|needinfo?(vondruch@redhat.c |
|om)
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
Pavel Valena changed:
What|Removed |Added
Flags||needinfo?(vondruch@redhat.c
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #6 from Pavel Valena ---
Hello,
sorry for the delay. Unfortunately, I forgot to continue to work on this, after
getting initially stuck with the test suite. Now I was able to debug it, and
all the tests pass.
I've a also fixed the
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #5 from Vít Ondruch ---
Ping?
--
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
___
packag
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #4 from Vít Ondruch ---
* Licensing
- Please check the license of
test/dummy/app/assets/stylesheets/erb_render_with_context.css.erb. It seems
the package should include the `OFL` license. But probably better to check
with ups
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #3 from Vít Ondruch ---
* Weird RexExp
- The 'g' in the following RegExp is probably not needed, since you are
matching single line anyway:
~~~
sed -i -e '/require .pry./ s/^/#/g' test/test_helper.rb
~~~
* Test suite
- It see
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
Vít Ondruch changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
Pavel Valena changed:
What|Removed |Added
Blocks||1871622
Referenced Bugs:
https://bu
https://bugzilla.redhat.com/show_bug.cgi?id=1883732
--- Comment #1 from Pavel Valena ---
Note that te test suite is troublesome at this moment. I didn't manage to make
the tests work, due to paths (assets not found, or even SassC). And CSS
'compressor' doesn't seem to work. I will try to resolv
14 matches
Mail list logo