<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/gitlab/gitlab-ce.git/lib/banzai/pipeline, branch ci-https</title>
<subtitle>gitlab.com: gitlab-org/gitlab-ce.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/'/>
<entry>
<title>Use a more powerful query to match videos in img tags</title>
<updated>2016-07-19T16:51:09+00:00</updated>
<author>
<name>Rémy Coutable</name>
<email>remy@rymai.me</email>
</author>
<published>2016-07-12T17:31:20+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=98e540532cc2706e4cdc027bd2acb8406e954ddc'/>
<id>98e540532cc2706e4cdc027bd2acb8406e954ddc</id>
<content type='text'>
Also, always add a link to download videos since video playback is
tricky. Also, it solves the issue with email client not supporting
videos.

Signed-off-by: Rémy Coutable &lt;remy@rymai.me&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Also, always add a link to download videos since video playback is
tricky. Also, it solves the issue with email client not supporting
videos.

Signed-off-by: Rémy Coutable &lt;remy@rymai.me&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>First support of videos in issues, MRs and notes</title>
<updated>2016-07-19T16:51:09+00:00</updated>
<author>
<name>Eric Hayes</name>
<email>eric@erichayes.net</email>
</author>
<published>2016-04-03T05:00:06+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=c266c7fa18848586cd6ee903cc4c4d4f9049100e'/>
<id>c266c7fa18848586cd6ee903cc4c4d4f9049100e</id>
<content type='text'>
* Registered video MIME types
* Currently supporting browser-supported formats with extensions that match the mime type
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
* Registered video MIME types
* Currently supporting browser-supported formats with extensions that match the mime type
</pre>
</div>
</content>
</entry>
<entry>
<title>Add blockquote fence syntax to Markdown</title>
<updated>2016-07-10T01:23:05+00:00</updated>
<author>
<name>Douwe Maan</name>
<email>douwe@selenight.nl</email>
</author>
<published>2016-04-27T21:38:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=6a477b9bfc5bf9b32c9a961269066694d1216dce'/>
<id>6a477b9bfc5bf9b32c9a961269066694d1216dce</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Enable Style/EmptyLines cop, remove redundant ones</title>
<updated>2016-07-01T19:56:17+00:00</updated>
<author>
<name>Grzegorz Bizon</name>
<email>grzesiek.bizon@gmail.com</email>
</author>
<published>2016-07-01T19:56:17+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=9e211091a85c20adea63b89111240350d6d8ffcb'/>
<id>9e211091a85c20adea63b89111240350d6d8ffcb</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Support for rendering/redacting multiple documents</title>
<updated>2016-06-24T09:46:39+00:00</updated>
<author>
<name>Yorick Peterse</name>
<email>yorickpeterse@gmail.com</email>
</author>
<published>2016-06-21T11:35:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=d470f3d1954a33d2ea6069fadcbb9810267b024f'/>
<id>d470f3d1954a33d2ea6069fadcbb9810267b024f</id>
<content type='text'>
This commit changes the way certain documents are rendered (currently
only Notes) and how documents are redacted. Previously both rendering
and redacting would run on a per document basis. The result of this was
that for every document we'd have to run countless queries just to
figure out if we could display a set of links or not.

This commit changes things around so that redacting Markdown documents
is no longer tied into the html-pipeline Gem. This in turn allows it to
redact multiple documents in a single pass, thus reducing the number of
queries needed.

In turn rendering issue/merge request notes has been adjusted to take
advantage of this new setup. Instead of rendering Markdown somewhere
deep down in a view the Markdown is rendered and redacted in the
controller (taking the current user and all that into account). This has
been done in such a way that the "markdown()" helper method can still be
used on its own.

This particular commit also paves the way for caching rendered HTML on
object level. Right now there's an accessor method Note#note_html which
is used for setting/getting the rendered HTML. Once we cache HTML on row
level we can simply change this field to be a column and call a "save"
whenever needed and we're pretty much done.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit changes the way certain documents are rendered (currently
only Notes) and how documents are redacted. Previously both rendering
and redacting would run on a per document basis. The result of this was
that for every document we'd have to run countless queries just to
figure out if we could display a set of links or not.

This commit changes things around so that redacting Markdown documents
is no longer tied into the html-pipeline Gem. This in turn allows it to
redact multiple documents in a single pass, thus reducing the number of
queries needed.

In turn rendering issue/merge request notes has been adjusted to take
advantage of this new setup. Instead of rendering Markdown somewhere
deep down in a view the Markdown is rendered and redacted in the
controller (taking the current user and all that into account). This has
been done in such a way that the "markdown()" helper method can still be
used on its own.

This particular commit also paves the way for caching rendered HTML on
object level. Right now there's an accessor method Note#note_html which
is used for setting/getting the rendered HTML. Once we cache HTML on row
level we can simply change this field to be a column and call a "save"
whenever needed and we're pretty much done.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix description and GFM pipelines conflicting</title>
<updated>2016-06-13T12:06:40+00:00</updated>
<author>
<name>Sean McGivern</name>
<email>sean@gitlab.com</email>
</author>
<published>2016-06-13T12:06:40+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=03d2bf141cde7bb12f88f25bcb08a612e65044c4'/>
<id>03d2bf141cde7bb12f88f25bcb08a612e65044c4</id>
<content type='text'>
Consider this command:

    bundle exec rails r "include GitlabMarkdownHelper
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;', pipeline: :description)
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;')"

And the same in the opposite order:

    bundle exec rails r "include GitlabMarkdownHelper
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;')
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;', pipeline: :description)"

Before this change, they would both output:

    &lt;p&gt;&lt;span&gt;this is a span&lt;/span&gt;&lt;/p&gt;
    &lt;p&gt;this is a span&lt;/p&gt;

That's because `span` is added to the list of whitelisted elements in
the `SanitizationFilter`, but this method tries not to make the same
changes multiple times. Unfortunately,
`HTML::Pipeline::SanitizationFilter::LIMITED`, which is used by the
`DescriptionPipeline`, uses the same Ruby objects for all of its hash
values _except_ `:elements`.

That means that whichever of `DescriptionPipeline` and `GfmPipeline` is
called first would have `span` in its whitelisted elements, and the
second wouldn't.

Fix this by creating an entirely separate hash, before either pipeline
is invoked.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Consider this command:

    bundle exec rails r "include GitlabMarkdownHelper
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;', pipeline: :description)
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;')"

And the same in the opposite order:

    bundle exec rails r "include GitlabMarkdownHelper
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;')
    puts markdown('&lt;span&gt;this is a span&lt;/span&gt;', pipeline: :description)"

Before this change, they would both output:

    &lt;p&gt;&lt;span&gt;this is a span&lt;/span&gt;&lt;/p&gt;
    &lt;p&gt;this is a span&lt;/p&gt;

That's because `span` is added to the list of whitelisted elements in
the `SanitizationFilter`, but this method tries not to make the same
changes multiple times. Unfortunately,
`HTML::Pipeline::SanitizationFilter::LIMITED`, which is used by the
`DescriptionPipeline`, uses the same Ruby objects for all of its hash
values _except_ `:elements`.

That means that whichever of `DescriptionPipeline` and `GfmPipeline` is
called first would have `span` in its whitelisted elements, and the
second wouldn't.

Fix this by creating an entirely separate hash, before either pipeline
is invoked.
</pre>
</div>
</content>
</entry>
<entry>
<title>Split Markdown rendering &amp; reference gathering</title>
<updated>2016-05-26T15:14:00+00:00</updated>
<author>
<name>Yorick Peterse</name>
<email>yorickpeterse@gmail.com</email>
</author>
<published>2016-05-26T11:16:43+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=86166d28029d5fcc729f7b7f5a41635c2e783a9e'/>
<id>86166d28029d5fcc729f7b7f5a41635c2e783a9e</id>
<content type='text'>
This splits the Markdown rendering and reference extraction phases into
two distinct code bases. The reference extraction phase no longer relies
on the html-pipeline Gem (and any related code) and allows for
extracting of references from multiple HTML nodes in a single pass. This
means that if you want to extract user references from 200 comments you
no longer need to run 200 times N number of queries, instead only a
handful of queries may be needed.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This splits the Markdown rendering and reference extraction phases into
two distinct code bases. The reference extraction phase no longer relies
on the html-pipeline Gem (and any related code) and allows for
extracting of references from multiple HTML nodes in a single pass. This
means that if you want to extract user references from 200 comments you
no longer need to run 200 times N number of queries, instead only a
handful of queries may be needed.
</pre>
</div>
</content>
</entry>
<entry>
<title>Create DiffFilter and change SystemNoteService#change_title to use Gitlab::Diff::InlineDiff</title>
<updated>2016-05-18T16:15:10+00:00</updated>
<author>
<name>Adam Butler</name>
<email>adam@littlebigmedia.co.uk</email>
</author>
<published>2016-04-06T19:37:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=8a8b5497c5142f225fb4fa3f5441526a0652869a'/>
<id>8a8b5497c5142f225fb4fa3f5441526a0652869a</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'patch/fix-markdown-preview-wikis' into 'master'</title>
<updated>2016-04-07T00:17:21+00:00</updated>
<author>
<name>Robert Speicher</name>
<email>robert@gitlab.com</email>
</author>
<published>2016-04-07T00:17:21+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=730625f022b1c3b9394cf944f04968a29ac8dc36'/>
<id>730625f022b1c3b9394cf944f04968a29ac8dc36</id>
<content type='text'>

Wiki preview URL converting problem [via Markdown]

Current implementation when rendering the preview, thinks relative links are for project repository files.

We are creating a new preview route that will define correct context data to render for wikis instead.

Fixes #2380, #1184

See merge request !3461</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>

Wiki preview URL converting problem [via Markdown]

Current implementation when rendering the preview, thinks relative links are for project repository files.

We are creating a new preview route that will define correct context data to render for wikis instead.

Fixes #2380, #1184

See merge request !3461</pre>
</div>
</content>
</entry>
<entry>
<title>Ensure correct filter order to validate with our markdown spec</title>
<updated>2016-04-06T06:30:56+00:00</updated>
<author>
<name>Gabriel Mazetto</name>
<email>gabriel@gitlab.com</email>
</author>
<published>2016-04-01T18:03:39+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=207b7218aa4394dc24c68041eade04474ff41537'/>
<id>207b7218aa4394dc24c68041eade04474ff41537</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
</feed>
