Skip to content

Conversation

@MariusBluem
Copy link
Member

Signed-off-by: Marius Blüm marius@lineone.io

Signed-off-by: Marius Blüm <marius@lineone.io>
@MariusBluem MariusBluem added the 3. to review Waiting for reviews label Mar 23, 2017
@mention-bot
Copy link

@MariusBluem, thanks for your PR! By analyzing the history of the files in this pull request, we identified @georgehrke, @tcitworld and @ChristophWurst to be potential reviewers.

@tcitworld
Copy link
Member

Isn't the lack of CDATA going to break multi-line ?

@MariusBluem
Copy link
Member Author

Thought it makes no difference here, because we are not using <, > or & 🤔

appinfo/info.xml Outdated
* 🙋 **Attendees!** Invite people to your events.
* 🙈 **We’re not reinventing the wheel!** Based on the great [davclient.js](https://github.com/evert/davclient.js), [ical.js](https://github.com/mozilla-comm/ical.js) and [fullcalendar](https://github.com/fullcalendar/fullcalendar) libraries.
]]</description>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're missing a > here, it should be ]]></description>.

Signed-off-by: Marius Blüm <marius@lineone.io>
@georgehrke georgehrke merged commit 9d2e46b into master Mar 24, 2017
@georgehrke georgehrke deleted the optimize-infoxml branch March 24, 2017 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants