Skip to content

Tkt 4 add task button#5

Open
p0stante wants to merge 4 commits intomasterfrom
tkt-4-add-task-button
Open

Tkt 4 add task button#5
p0stante wants to merge 4 commits intomasterfrom
tkt-4-add-task-button

Conversation

@p0stante
Copy link
Member

No description provided.

<PersistentState>
<option name="values">
<map>
<entry key="url" value="jar:file:/C:/Program%20Files/Android/Android%20Studio/plugins/android/lib/android.jar!/images/material_design_icons/content/ic_add_black_24dp.xml" />
Copy link
Contributor

Choose a reason for hiding this comment

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

If I get it right, this file is not supposed to be version controlled: absolute path will definitely be the source of pain.

Also, Ganttproject_mobile/.idea/caches/build_file_checksums.ser doesn't seem to be useful for VCS purposes

@@ -0,0 +1,5 @@
package com.bardsoftware.ganttproject_mobile;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add copyright notice header to all Java source code files:

/*
Copyright 2018 BarD Software s.r.o, Alexandra Toropova

This file is part of GanttProject, an opensource project management tool.

GanttProject is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
 the Free Software Foundation, either version 3 of the License, or
 (at your option) any later version.

GanttProject is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
GNU General Public License for more details.

You should have received a copy of the GNU General Public License
along with GanttProject.  If not, see <http://www.gnu.org/licenses/>.
*/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants