-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#16 - code style violations in headers #17
Conversation
Would be great to remove the inlined configuration after jcabi/jcabi-parent#19 is fixed.
Many thanks for the PR, let me find a reviewer for it |
@dmarkov any progress with that? |
@carlosmiranda please review this one |
<!-- | ||
@todo #16 Remove this declaration when parent pom would | ||
fix issue jcabi/jcabi-parent#19 | ||
--> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's align this line with line 102 above.
@maurezen One minor comment. |
@carlosmiranda fixed. |
@rultor can we merge? |
@carlosmiranda Your
This page explains how to configure it properly. The YAML format is explained here. |
@rultor I fixed config, try to merge again |
@yegor256 This is how I understand the <?xml version="1.0" encoding="UTF-8"?>
<p>
<entry key="assets">
<entry key="settings.xml">yegor256/home#assets/jcabi/settings.xml</entry>
<entry key="pubring.gpg">yegor256/home#assets/pubring.gpg</entry>
<entry key="secring.gpg">yegor256/home#assets/secring.gpg</entry>
<entry key="s3cfg">yegor256/home#assets/s3cfg</entry>
</entry>
<entry key="env">
<entry key="MAVEN_OPTS">-XX:MaxPermSize=256m -Xmx1g</entry>
</entry>
<entry key="install">
<item>sudo gem install pdd</item>
</entry>
<entry key="architect">
<item>yegor256</item>
<item>dmarkov</item>
</entry>
<entry key="merge">
<entry key="script">mvn clean install -Pqulice --errors --settings ../settings.xml
mvn clean
pdd --source=$(pwd) --verbose --file=/dev/null
</entry>
</entry>
<entry key="deploy">
<entry key="script">mvn clean deploy -Pqulice --errors --settings ../settings.xml
mvn clean
pdd --source=$(pwd) --verbose --file=jcabi-manifests.xml
s3cmd --no-progress put jcabi-manifests.xml --config=../s3cfg s3://pdd.teamed.io/jcabi-manifests.xml
</entry>
</entry>
<entry key="release">
<entry key="script">if [ -n ${snapshot} ]; then
echo 'Please specify next snapshot version, e.g. @rultor release, tag=`2.0`, snapshot=`2.1-SNAPSHOT`'
exit 1
fi
mvn versions:set "-DnewVersion=${tag}"
sed -i "s|<snapshotVersion>.*</snapshotVersion>|<snapshotVersion>${nextSnapshotVersion}</snapshotVersion>|" pom.xml
git commit -am "${tag}"
mvn clean deploy -Pqulice -Psonatype -Pjcabi --errors --settings ../settings.xml
mvn clean site-deploy -Psite --errors --settings ../settings.xml</entry>
</entry>
</p> This page explains how to configure it. |
@rultor merge it! :) |
@@ -99,4 +99,27 @@ | |||
</plugin> | |||
</plugins> | |||
</build> | |||
<!-- | |||
@todo #16 Remove this declaration when parent pom would | |||
fix issue jcabi/jcabi-parent#19 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add a single space here (f
in line 104 should be aligned with t
, not @
).
@maurezen rultor is complaining about puzzle format. Let's fix it (see my latest code comment above) |
@carlosmiranda done. |
@rultor Try the merge again please |
@carlosmiranda Thanks for your request. @yegor256 Please confirm this. |
@rultor merge it |
@carlosmiranda done, I added 20 mins in payment added +20 to your rating, now it is equal to +1750 |
@rultor deploy pls |
Added license.txt to configuration. Please also note jcabi/jcabi-parent#19