Description:
Tweak warning levels and remove .java-version, which was misleading.
Commit status:
[Not Reviewed]
References:
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Add another comment

r28:82a5bd5a2ac8 -

@@ -1,5 +1,5
1 (defproject project-checkup "0.1.4"
1 (defproject project-checkup "0.1.5"
2 :description "Makes sure your project is healthy. "
2 :description "Makes sure your project is healthy. :)"
3 :url "https://bitbucket.org/alysbrooks/project-checkup"
3 :url "https://bitbucket.org/alysbrooks/project-checkup"
4 :license {:name "Eclipse Public License"
4 :license {:name "Eclipse Public License"
5 :url "http://www.eclipse.org/legal/epl-v10.html"}
5 :url "http://www.eclipse.org/legal/epl-v10.html"}
@@ -105,13 +105,13
105 {:name "Project has a README"
105 {:name "Project has a README"
106 :function check-readme
106 :function check-readme
107 :description "Readme exists"
107 :description "Readme exists"
108 :level :suggestion
108 :level :warning
109 :follow-up "Add a README." }
109 :follow-up "Add a README." }
110 {:name "Project has a CHANGELOG"
110 {:name "Project has a CHANGELOG"
111 :function check-changelog
111 :function check-changelog
112 :description "Changelog exists"
112 :description "Changelog exists"
113 :level :suggestion
113 :level :warning
114 :follow-up "Add a CHANGELOG. Consider refering to keepachangelog.com" }
114 :follow-up "Add a CHANGELOG. Consider refering to keepachangelog.com for guidance." }
115 {:name "README has no placeholders"
115 {:name "README has no placeholders"
116 :function check-readme-placeholders
116 :function check-readme-placeholders
117 :description "No placeholders in README"
117 :description "No placeholders in README"
@@ -122,7 +122,7
122 :function check-license
122 :function check-license
123 :description "Project has a LICENSE file."
123 :description "Project has a LICENSE file."
124 :level :warning ;going with warning because a project might not have a license before release.
124 :level :warning ;going with warning because a project might not have a license before release.
125 :follow-up "Add a license to LICENSE. Consider using https://choosealicense.com/ if you need guidance. " }])
125 :follow-up "Add a license to LICENSE. Consider using https://choosealicense.com/ for guidance." }])
126
126
127 (defn perform-check [check project]
127 (defn perform-check [check project]
128 (let [{check-name :name function :function follow-up :follow-up
128 (let [{check-name :name function :function follow-up :follow-up
1 NO CONTENT: file was removed
NO CONTENT: file was removed
You need to be logged in to leave comments. Login now