<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>It's ok for me except for&nbsp;<a href="https://issues.jboss.org/browse/JDF-436">https://issues.jboss.org/browse/JDF-436</a> and minors remarks on the doc bellow.</div><div><br></div><div>In&nbsp;<a href="https://github.com/jboss-jdf/qstools/blob/master/README.md">https://github.com/jboss-jdf/qstools/blob/master/README.md</a>, there is no info about &nbsp;.quickstarts_ignore and the link in Plugin Documentation section is broken.</div><div><br></div><div>I sent you a PR to add fuse rules so you can check what I'm using.</div><div><br></div><div><br></div><div>Antoine</div><div><br></div><br><div><div>Le 24 juil. 2013 à 01:05, Rafael Benevides &lt;<a href="mailto:benevides@redhat.com">benevides@redhat.com</a>&gt; a écrit :</div><br class="Apple-interchange-newline"><blockquote type="cite">
  
    <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
  
  <div bgcolor="#FFFFFF" text="#000000">
    I'm pleased to announce the first release candidate of the 1.2
    series of QSTools. - I expect to have no more false violations or
    "you can ignore that violation".<br>
    <br>
    It is now much more flexible than the 1.0 series. The configuration
    is now centralized on&nbsp; github repo: <br>
    Check it out the possible configurations:
<a class="moz-txt-link-freetext" href="https://github.com/jboss-jdf/qstools/blob/master/config/qstools_config.yaml">https://github.com/jboss-jdf/qstools/blob/master/config/qstools_config.yaml</a><br>
    <br>
    We can now add different configuration rules for each quickstarts
    (as it was made for jpp quickstarts and it is probable that we will
    have to add fuse, drools and spring quickstarts specific rules, very
    soon) - they will be added on demand.<br>
    <br>
    Main changes from 1.0:<br>
    - Different rules for different quickstarts <br>
    - Configuration centralized online.<br>
    - Enhancement of Readmechecker - It now also checks the if the
    Readme metadata values are the expected values for tags like "Target
    Product", Level, etc.<br>
    - Enhancement of BomVersionChecker - You can now define a expected
    bom version instead of using stacks (useful for upstream projects)<br>
    - Enhancement of MavenCentralRepositoryChecker to permit skiping of
    this checker (useful for project quickstarts) -
    <a class="moz-txt-link-freetext" href="https://issues.jboss.org/browse/JDF-433">https://issues.jboss.org/browse/JDF-433</a><br>
    - Enhancement of the QSTools to read the .quickstarts_ignore file
    and ignore those quickstarts (useful for top level folders)<br>
    - Created a PomNameChecker that checks the pom name against the
    pattern defined at <a class="moz-txt-link-freetext" href="https://issues.jboss.org/browse/JDF-430">https://issues.jboss.org/browse/JDF-430</a><br>
    <br>
    You can try it running&nbsp; mvn
    org.jboss.maven.plugins:maven-qstools-plugin:1.2.0.CR1:check <br>
    <br>
    Antoine,<br>
    <br>
    Can you check if the behavior of this version under fuse
    quickstarts?<br>
    <br>
    If you find some bug, please let me know.<br>
    <br>
    Thanks<br>
    <br>
    <div class="moz-cite-prefix">Em 17/07/13 16:43, Rafael Benevides
      escreveu:<br>
    </div>
    <blockquote cite="mid:51E6F3E3.505@redhat.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
      Amazing job, Sande!<br>
      <br>
      Thanks. It was more than I expected. <br>
      <br>
      I saw your questions and I'll review it as I get close to modify
      QSTools. <br>
      <br>
      I'm working on Drools examples and I'd like to have QSTools to
      make the job for me. So it is probable that I'll update it&nbsp;
      (depending on the necessary effort) to have it prepared to handle
      quickstarts from different sources and use it on Drools examples.<br>
      <br>
      Thanks once more<br>
      <br>
      <br>
      <div class="moz-cite-prefix">Em 17/07/13 14:52, Sande Gilda
        escreveu:<br>
      </div>
      <blockquote cite="mid:51E6D9CB.1080201@redhat.com" type="cite">
        <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
        My first cut at modifying the QSTools to verify by product is
        under "
        <meta http-equiv="content-type" content="text/html;
          charset=ISO-8859-1">
        Modify the QSTools to use Product Specific Requirements" here: <a moz-do-not-send="true" href="https://docspace.corp.redhat.com/docs/DOC-132902#modify-the-qstools-to-use-product-specific-requirements">https://docspace.corp.redhat.com/docs/DOC-132902</a><br>
        <br>
        Feedback and suggestions are welcome!<br>
        <br>
        <div class="moz-cite-prefix">On 07/15/2013 08:51 PM, Rafael
          Benevides wrote:<br>
        </div>
        <blockquote cite="mid:51E4991A.6010307@redhat.com" type="cite">
          <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
          <br>
          <div class="moz-cite-prefix">Em 15/07/13 20:54, Sande Gilda
            escreveu:<br>
          </div>
          <blockquote cite="mid:51E48BB9.1000307@redhat.com" type="cite">
            <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
            <br>
            <div class="moz-cite-prefix">On 07/15/2013 06:18 PM, Rafael
              Benevides wrote:<br>
            </div>
            <blockquote cite="mid:51E47525.1040606@redhat.com" type="cite">
              <meta http-equiv="content-type" content="text/html;
                charset=ISO-8859-1">
              Hi all, Sande and Pete,<br>
              <br>
              One significant change in JDF Quickstarts repo is the use
              of git submodules to bring remote quickstarts to JDF.
              But... Sometimes remote quickstarts doesn't ( and don't
              want/need to ) follow JDF Contributing guide ( <a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://github.com/jboss-jdf/jboss-as-quickstart/blob/master/CONTRIBUTING.md">https://github.com/jboss-jdf/jboss-as-quickstart/blob/master/CONTRIBUTING.md</a>
              ).<br>
              <br>
              There are some requirements from QSTools ( <a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://docspace.corp.redhat.com/docs/DOC-132902">https://docspace.corp.redhat.com/docs/DOC-132902</a>
              ) that I believe that we should update to split in two
              categories ( desired and mandatory ).<br>
              <br>
              The definitions bellow are what I see differences across
              JBoss projects:<br>
              &nbsp;- package and groupId name (of course) - We already
              defined that using org.jboss.quickstarts.(eap|wfk|...) is
              optional from other Quickstarts (not JDF) but should be
              consistent within the product<br>
            </blockquote>
            Agreed. Could we define properties or some other type of
            file that could define the valid packages, groups, etc for
            each product?<br>
          </blockquote>
          <br>
          Yes. That's Pete's suggestion. We could keep this definition
          file on QStools github repo. I thought in a yaml format to
          keep it.<br>
          <br>
          Sande, Can you edit the QSTools requirement docspace to define
          what should be a "per product" Checker ? Nobody other than you
          is the best to provide this definition. I understand that what
          will not be a "per product" Checker, it should be a mandatory
          instruction.<br>
          <br>
          With this in hand I can start a QSTools refactoring. I was
          wondering that a "per product" violation is a "warning" level
          violation and I'll sign it on QSTools report with a yellow
          color. In a mandatory violation I'll sign it with a red color.
          <br>
          <br>
          I'm trying to make QSTools a tooling to help us and it should
          be update as we need. But recently, the reported violations
          seems more a barrier than a gate.<br>
          <br>
          Pete,<br>
          <br>
          Any objections ?<br>
          <blockquote cite="mid:51E48BB9.1000307@redhat.com" type="cite">
            <blockquote cite="mid:51E47525.1040606@redhat.com" type="cite"> &nbsp;- License Headers<br>
            </blockquote>
            Yes. We saw this with the Spring-based quickstarts that
            originate elsewhere. I'd still like to see this reported in
            case they are EAP quickstarts.<br>
            <blockquote cite="mid:51E47525.1040606@redhat.com" type="cite"> &nbsp;-
              <meta charset="utf-8">
              Spacing and Indentation formats<br>
              <br>
            </blockquote>
            I don't see this as being something someone would object
            too. But maybe I'm wrong? Again, I'd still like to see this
            reported in case they are EAP quickstarts.<br>
            <br>
          </blockquote>
          One example: The Infinispan project is the one who uses a
          different format. They use 3-space for indentation.<br>
          <br>
          <blockquote cite="mid:51E48BB9.1000307@redhat.com" type="cite">
            <blockquote cite="mid:51E47525.1040606@redhat.com" type="cite"> What do you think? Is it it desired to be
              more or less restrictive for other quickstarts and also
              turn it in an automated pattern?<br>
              <br>
              I'm bringing this discussion mainly because it is a
              recurrent discussion for remote projects like <br>
              - Infinispan: <a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://github.com/infinispan/jdg-quickstart/pull/20#issuecomment-20968520">https://github.com/infinispan/jdg-quickstart/pull/20#issuecomment-20968520</a><br>
              - GateIn: <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://transcripts.jboss.org/channel/irc.freenode.org/%23jboss-jdf/2013/%23jboss-jdf.2013-06-21.log.html#t2013-06-21T13:39:31">http://transcripts.jboss.org/channel/irc.freenode.org/%23jboss-jdf/2013/%23jboss-jdf.2013-06-21.log.html#t2013-06-21T13:39:31</a><br>
              - And probable new others like BRMS, Fuse and Switchyard
              Quickstarts.<br>
            </blockquote>
          </blockquote>
          - Adding Spring Quickstarts to the list :)<br>
          <blockquote cite="mid:51E48BB9.1000307@redhat.com" type="cite">
            <blockquote cite="mid:51E47525.1040606@redhat.com" type="cite"> <br>
              <meta charset="utf-8">
              <meta charset="utf-8">
              <pre class="moz-signature" cols="72">-- 
Rafael Benevides | Senior Software Engineer
Red Hat Brazil
+55-61-9269-6576

Better technology. Faster innovation. Powered by community collaboration.
See how it works at <a href="http://redhat.com">redhat.com</a> </pre>
            </blockquote>
            <br>
          </blockquote>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
jdf-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:jdf-dev@lists.jboss.org">jdf-dev@lists.jboss.org</a>
<a class="moz-txt-link-freetext" href="https://lists.jboss.org/mailman/listinfo/jdf-dev">https://lists.jboss.org/mailman/listinfo/jdf-dev</a></pre>
    </blockquote>
    <br>
  </div>

</blockquote></div><br></body></html>